ankidroid / ankidroid/Anki-Android

lint: Enable unused import checks

Offen
#21,208 4 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
Blocked by dependency Keep Open
Vorherrschende Sprache
Kotlin
Sterne
11.8k
Forks
2.9k
Ø Merge
2 T. 3 Std.
Gemergte PRs (30 T.)
171

Beschreibung

These were disabled in ktlint 1.7 and is now opt-in. `ktlint` planned to remove these checks, but this decision has been reversed

> Ktlint analyses code on a file by file base. Meaning, that all linting/formatting decisions have to be made based on information in the file that is being processed. Information in other files is not taken into account at all. For a 100% reliable detection whether an import is used, or unused, a file needs to be analyzed within the scope of the project and its dependencies.
>
> The no-unused-imports rule is marked as deprecated starting from Ktlint 1.7.0, and will be removed in Ktlint 2.0.0. Until the rule is removed, it can be used by explicitly enabling the rule in the .editorconfig, by adding:

* https://redirect.github.com/pinterest/ktlint/issues/3038

----

I will use `detekt` for this: https://detekt.dev/

I have been planning to enable `detekt` for a while, and this is a sufficient reason to start. Detekt bugfixes should be added as TODOs

Beitragsleitfaden

Beitragsleitfaden öffnen

Bewertung

Dieses Issue wurde noch nicht bewertet.

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.