Include Roslynator.Analyzers and Roslynator.CodeFixes with more strict default severity levels
Les mainteneurs répondent en général sous 1 jour
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Accessibilité débutants
- 38/100
Piste de recherche
Commencez par examiner les normes C# existantes du dépôt ainsi que la configuration de editorconfig, puis comparez-les aux analyseurs et refactorings Roslynator listés. Le travail est considéré comme terminé lorsque Roslynator est inclus et que les règles RCS sélectionnées disposent de niveaux de gravité par défaut documentés, avec les ajouts correspondants aux normes C# lorsque nécessaire.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Introduce https://github.com/JosefPihrt/Roslynator, which has 500+ analyzers, and also a large number of refactorings.
Lists of each:
http://pihrt.net/Roslynator/Analyzers
http://pihrt.net/Roslynator/Refactorings
Here's a very preliminary set of some recommendations for default severities. This is by no means exhaustive and is only the set that I came up with upon looking at the Info level severity issues from a single project I'm working on at the moment.
For some of these that aren't already covered, corresponding rules should be added to the C# standards.
# RCS1001: Add braces (when expression spans over multiple lines).
dotnet_diagnostic.RCS1001.severity = error
# RCS1003: Add braces to if-else (when expression spans over multiple lines).
dotnet_diagnostic.RCS1003.severity = error
# RCS1007: Add braces
dotnet_diagnostic.RCS1007.severity = error
# RCS1044: Remove original exception from throw statement
dotnet_diagnostic.RCS1044.severity = error
# RCS1123: Add parentheses according to operator precedence.
dotnet_diagnostic.RCS1123.severity = error
# RCS1037: Remove trailing white-space.
dotnet_diagnostic.RCS1037.severity = none
# RCS1036: Remove redundant empty line.
dotnet_diagnostic.RCS1036.severity = none
# RCS1170: Use read-only auto-implemented property.
dotnet_diagnostic.RCS1170.severity = warning
# RCS1213: Remove unused member declaration.
dotnet_diagnostic.RCS1213.severity = warning
- Langage dominant
- C#
- Étoiles
- 12
- Forks
- 16
- Merge moyen
- 5 min
- PR mergées (30 j)
- 12
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Aucun modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de IntelliTect/CodingGuidelines
-
Overriding ToString()Ouverte
Difficulté 3/5 1-2 jours Accessibilité débutants 25/100
IntelliTect/CodingGuidelines#290 ·
Les mainteneurs répondent en général sous 1 jour
-
proposal: ban List<T>.ForEachOuverte
Difficulté 2/5 1-3 heures Accessibilité débutants 45/100
IntelliTect/CodingGuidelines#253 ·
Les mainteneurs répondent en général sous 1 jour
-
Review naming recommendation for fieldsPeut-être à nouveau libre @MarkMichaelis l’a pris il y a 1334 jours, et aucune pull request n’est ouverte. Ouverte.editorconfig C# coding guidelines proposal
IntelliTect/CodingGuidelines#249 · 6 commentaires · 1 réaction · 2 personnes assignées ·
Les mainteneurs répondent en général sous 1 jour
-
INTL0001 when using recordsOuverteanalyzer C# coding guidelines proposal
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 35/100
IntelliTect/CodingGuidelines#231 ·
Les mainteneurs répondent en général sous 1 jour
-
bug
Difficulté 3/5 1-2 jours Accessibilité débutants 35/100
IntelliTect/CodingGuidelines#230 ·
Les mainteneurs répondent en général sous 1 jour
Toutes les issues de IntelliTect/CodingGuidelines
Issues similaires
-
needs-area-label
Difficulté 2/5 1-3 heures Accessibilité débutants 82/100
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 76/100
Fallout-build/Fallout#708 ·
Les mainteneurs répondent en général sous 5 jours
-
bug next version
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
Rem0o/FanControl.Releases#4220 · 1 commentaire ·
-
area:repo audit:approved audit:auto priority:P3
Difficulté 2/5 1-3 heures Accessibilité débutants 66/100
elsa-workflows/elsa-core#8771 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
nkdAgility/azure-devops-migration-tools#3217 ·
Les mainteneurs répondent en général sous 1 jour