dotnet/runtime

Enable more style checking analyzers

Open

#78.414 aperta il 15 nov 2022

Vedi su GitHub
 (17 commenti) (0 reazioni) (0 assegnatari)C# (5445 fork)batch import
area-Metahelp wanted

Metriche repository

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

Descrizione

There are several analyzers that check for code style that runtime repo could benefit from if enabled. They are not too pedantic and could help to make the large codebase more readable.

For example

  • #94827
  • SA1107: Code should not contain multiple statements on one line
  • SA1119: A C# statement contains parenthesis which are unnecessary and should be removed.
  • SA1123: Region should not be located within a code element
  • SA1128: Put constructor initializers on their own line
  • SA1130: Use lambda syntax
  • #94098
  • SA1507: Code should not contain multiple blank lines in a row

Guida contributor

Direzione di ricerca
Scegli un elemento non selezionato dall'elenco (ad esempio SA1107), abilita l'analizzatore corrispondente nel progetto e correggi eventuali violazioni in tutto il codebase.
Tech stack
csharp
Dominio
backend
Tipo issue
Manutenzione
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
C#Git
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75