unoplatform/uno

[Breaking] `SetterBase.set_Property` should be removed

Open

#13.050 aperta il 29 lug 2023

Vedi su GitHub
 (2 commenti) (0 reazioni) (0 assegnatari)C# (837 fork)batch import
difficulty/starter 🚀good first issuekind/bugno-repro

Metriche repository

Star
 (9789 star)
Metriche merge PR
 (Merge medio 14g 8h) (130 PR mergiate in 30 g)

Descrizione

Current behavior

https://github.com/unoplatform/uno/blob/709dc45ba2e11be607714d5508dc16dc6bf8c41c/src/Uno.UI/UI/Xaml/SetterBase.cs#L18-L23

Expected behavior

No response

How to reproduce it (as minimally and precisely as possible)

No response

Workaround

No response

Works on UWP/WinUI

None

Environment

No response

NuGet package version(s)

No response

Affected platforms

No response

IDE

No response

IDE version

No response

Relevant plugins

No response

Anything else we need to know?

No response

Guida contributor

Direzione di ricerca
Il problema richiede la rimozione del metodo SetterBase.set Property come mostrato nelle righe 18-23 di src/Uno.UI/UI/Xaml/SetterBase.cs. Il repository ha commenti (2) che possono fornire contesto. La modifica è breaking, quindi tutti gli utilizzi di set Property nell'intera codebase devono essere identificati e aggiornati o rimossi. Il comportamento atteso non è descritto, quindi il contributore dovrebbe prima confermare l'esatta sostituzione con i maintainer. Il problema è stale, quindi considera di coinvolgere la comunità prima di procedere.
Tech stack
csharp
Dominio
frontend
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
C#
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60