Include Roslynator.Analyzers and Roslynator.CodeFixes with more strict default severity levels
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 38/100
Línea de trabajo
Empieza revisando los estándares de C# existentes en el repositorio y la configuración de editorconfig; después, compáralos con los analizadores y refactorizaciones de Roslynator enumerados. Se considera terminado cuando Roslynator está incluido y las reglas RCS seleccionadas tienen severidades predeterminadas documentadas, con las adiciones correspondientes a los estándares de C# cuando sea necesario.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
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
- Lenguaje dominante
- C#
- Estrellas
- 12
- Forks
- 16
- Merge medio
- 2 min
- PR fusionados (30 d)
- 7
Guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de IntelliTect/CodingGuidelines
-
Overriding ToString() Abierto
Dificultad 3/5 1-2 días Aptitud para principiantes 25/100
IntelliTect/CodingGuidelines#290 ·
-
proposal: ban List<T>.ForEach Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 45/100
IntelliTect/CodingGuidelines#253 ·
-
.editorconfig C# coding guidelines proposal
IntelliTect/CodingGuidelines#249 · 6 comentarios · 1 reacción · 2 asignados ·
-
INTL0001 when using records Abiertoanalyzer C# coding guidelines proposal
Dificultad 5/5 Más de una semana Aptitud para principiantes 35/100
IntelliTect/CodingGuidelines#231 ·
-
bug
Dificultad 3/5 1-2 días Aptitud para principiantes 35/100
IntelliTect/CodingGuidelines#230 ·
Todos los issues de IntelliTect/CodingGuidelines
Issues similares
-
Documentation
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
cake-build/cake#5024 ·
-
Gå gjennom ESLint-suppressions AbiertoFrontend status/draft TechnicalDebt
Dificultad 2/5 1-2 días Aptitud para principiantes 75/100
Altinn/altinn-auth#4143 ·
-
.NET Flaky Test Testing Tests
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
getsentry/sentry-dotnet#5617 · 1 comentario ·
-
Add more to the documentation Abierto:watch: Not Triaged dotnet-fsharp/svc
Dificultad 1/5 Menos de una hora Aptitud para principiantes 90/100
-
Client customer-reported needs-team-attention question Service Attention WebPubSub
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
Azure/azure-sdk-for-net#63292 · 3 comentarios · 1 reacción ·