dotnet/runtime

Enable more style checking analyzers

Open

#78.414 aberto em 15 de nov. de 2022

Ver no GitHub
 (17 comments) (0 reactions) (0 assignees)C# (5.445 forks)batch import
area-Metahelp wanted

Métricas do repositório

Stars
 (17.886 stars)
Métricas de merge de PR
 (Mesclagem média 12d 11h) (661 fundiu PRs em 30d)

Description

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

Guia do colaborador

Direção de pesquisa
Escolha um item não marcado da lista (por exemplo, SA1107), ative o analisador correspondente no projeto e corrija todas as violações em todo o codebase.
Pilha de tecnologia
csharp
Domain
backend
Tipo Issue
Manutenção
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
C#Git
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
75