dotnet/roslyn

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

Open

#61 860 ouverte le 13 juin 2022

Voir sur GitHub
 (0 commentaires) (1 réaction) (0 assignés)C# (4 257 forks)batch import
Area-IDEFeature - Editor Config UIhelp wanted

Métriques du dépôt

Stars
 (20 414 stars)
Métriques de merge PR
 (Merge moyen 6j 17h) (256 PRs mergées en 30 j)

Description

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.

Guide contributeur

Direction de recherche
Examinez le fichier lié CommonCodeStyleSettingsProvider.cs pour identifier les options sans stockage EditorConfig. Concevez ensuite un mécanisme de validation pour garantir que toutes les futures options de style de code aient un stockage EditorConfig. Mettez à jour le code en conséquence et ajoutez des tests.
Stack technique
csharp
Domaine
developer experiencetooling
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
GitC#
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70