Support ktlint's suppression style in detekt-formatting

(0 commenti) (3 reazioni) (0 assegnatari)Kotlin (834 fork)batch import

formattinghelp wantedimprovement

Metriche repository

Star: (6942 stelle)
Metriche merge PR: (Merge medio 4g 9h) (65 PR mergiate in 30 g)

Descrizione

Expected Behavior

ktlint suppressions are respected by detekt.

Current Behavior

ktlint's suppressions aren't respected by detekt.

Support should cover these formats: https://github.com/pinterest/ktlint/blob/ae796b60f9340492e12a720d514e9646a1e9d068/documentation/snapshot/docs/faq.md#how-do-i-suppress-errors-for-a-lineblockfile

Suppression types:

@Suppress("ktlint") suppresses all ktlint rules (we could extend this so we could do the same thing with our rule sets too?)
@Suppress("ktlint:rule-set-id:rule-id") suppresses a specific rule

Context

If someone moves from ktlint to detekt and uses ktlint suppressions they currently need to adjust them for detekt. It should be easier to move between detekt and ktlint & its wrappers.

Guida contributor

Direzione di ricerca: Indaga sul modulo detekt formatting, in particolare su come i set di regole sono attualmente soppressi. Esamina la documentazione del formato di soppressione di ktlint per comprenderne i pattern. Identifica nel codebase dove detekt gestisce attualmente le soppressioni (probabilmente nella fase di parsing o reporting delle regole). Proponi una modifica per estendere il meccanismo di soppressione in modo che riconosca anche le annotazioni in stile ktlint. Il problema fa riferimento a una pagina FAQ di ktlint; esaminala e considera come mappare gli identificatori delle regole di ktlint ai set di regole di detekt.
Tech stack: kotlin
Dominio: backend
Tipo issue: Funzionalità
Difficoltà: 3
Tempo stimato: Mezza giornata
Stato attività: Attiva
Chiarezza: Chiara
Prerequisiti: Kotlindetektktlint
Adatta ai principianti: 70