saveourtool/diktat

WhiteSpaceRule doesn't cover some cases with comma

Aberta

#1.239 aberto em 21 de mar. de 2022

Ver no GitHubGuia do colaborador
 (2 comentários) (0 reação) (1 responsável)Kotlin (40 forks)github user discovery
enhancementgood first issuehelp wantedneeded

Métricas do repositório

Stars
 (571 estrelas)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

Description

I have found a reproducer for https://github.com/analysis-dev/diktat/blob/5e2e2ed194a1f74904893ecd8a4c7dd4bb9afd57/diktat-rules/src/main/kotlin/org/cqfn/diktat/ruleset/rules/chapter3/files/WhiteSpaceRule.kt#L254-L255

[WARN ] 2022-03-21 18:01:16 Colon with treeParent.elementType=DESTRUCTURING_DECLARATION_ENTRY, not handled by WhiteSpaceRule
[WARN ] 2022-03-21 18:01:16 Colon with treeParent.elementType=DESTRUCTURING_DECLARATION_ENTRY, not handled by WhiteSpaceRule
[WARN ] 2022-03-21 18:01:17 Colon with treeParent.elementType=DESTRUCTURING_DECLARATION_ENTRY, not handled by WhiteSpaceRule
[WARN ] 2022-03-21 18:01:17 Colon with treeParent.elementType=DESTRUCTURING_DECLARATION_ENTRY, not handled by WhiteSpaceRule

However, I don't even know which file has caused this :(

Guia do colaborador

Direção de pesquisa
Examine o arquivo WhiteSpaceRule.kt, especificamente a parte que lida com tokens de dois pontos (por volta da linha 254). Adicione tratamento para DESTRUCTURING DECLARATION ENTRY como tipo pai para suprimir avisos.
Pilha de tecnologia
kotlin
Domain
tooling
Tipo Issue
Bug
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
KotlinGit
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
75