saveourtool/diktat

WhiteSpaceRule doesn't cover some cases with comma

Ouverte

#1 239 ouverte le 21 mars 2022

Voir sur GitHubGuide contributeur
 (2 commentaires) (0 réaction) (1 personne assignée)Kotlin (40 forks)github user discovery
enhancementgood first issuehelp wantedneeded

Métriques du dépôt

Stars
 (571 étoiles)
Métriques de merge PR
 (Aucune PR mergée en 30 j)

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 :(

Guide contributeur

Direction de recherche
Examinez le fichier WhiteSpaceRule.kt, en particulier la partie qui traite les jetons de deux points (autour de la ligne 254). Ajoutez la gestion de DESTRUCTURING DECLARATION ENTRY en tant que type parent pour supprimer les avertissements.
Stack technique
kotlin
Domaine
tooling
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.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
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
KotlinGit
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75