dotnet/runtime

Sort object properties optimally in emitted config binding gen logic

Open

#92.594 aperta il 25 set 2023

Vedi su GitHub
 (3 commenti) (0 reazioni) (0 assegnatari)C# (5445 fork)batch import
area-Extensions-Configurationhelp wantedtenet-performance

Metriche repository

Star
 (17.886 star)
Metriche merge PR
 (Merge medio 12g 11h) (661 PR mergiate in 30 g)

Descrizione

Per @eerhardt in https://github.com/dotnet/runtime/pull/89587.

ints are pretty common in config. It might make sense to keep this near the top of the check, so we don't have to check a bunch of other (less common) types first.

@dotnet/area-extensions-configuration.

Guida contributor

Direzione di ricerca
Esamina l'ordine attuale dei controlli dei tipi di proprietà nel codice di binding della configurazione emesso e proponi un riordino che posizioni i tipi comuni come int vicino all'inizio per ridurre i controlli non necessari.
Tech stack
csharp
Dominio
backend
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
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.
Abbastanza chiara
Prerequisiti
C#Config binding
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60