dotnet/roslyn

EditorConfig UI shouldn't include roaming-only code-style options

Open

#61.860 aperta il 13 giu 2022

Vedi su GitHub
 (0 commenti) (1 reazione) (0 assegnatari)C# (4257 fork)batch import
Area-IDEFeature - Editor Config UIhelp wanted

Metriche repository

Star
 (20.414 star)
Metriche merge PR
 (Merge medio 6g 17h) (256 PR mergiate in 30 g)

Descrizione

All code-style options in EditorConfig UI should have editorconfig storage location. For example, the following shouldn't exist:

https://github.com/dotnet/roslyn/blob/eee6b3c9180c77f4d58be0b2f8cafd2204297d10/src/VisualStudio/Core/Def/EditorConfigSettings/DataProvider/CodeStyle/CommonCodeStyleSettingsProvider.cs#L135-L138

We need a way to ensure this doesn't happen again. So the fix isn't simply to delete the above line. We also want to ensure that all the options are up to date.

Guida contributor

Direzione di ricerca
Esamina il file collegato CommonCodeStyleSettingsProvider.cs per identificare le opzioni senza storage EditorConfig. Quindi progetta un meccanismo di validazione per assicurarsi che tutte le future opzioni di stile del codice abbiano lo storage EditorConfig. Aggiorna il codice di conseguenza e aggiungi test.
Tech stack
csharp
Dominio
developer experiencetooling
Tipo issue
Bug
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.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GitC#
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70