Skip to content

Integrate detekt #5135

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
TimoPtr opened this issue Mar 20, 2025 · 0 comments · May be fixed by #5312
Open

Integrate detekt #5135

TimoPtr opened this issue Mar 20, 2025 · 0 comments · May be fixed by #5312
Labels
enhancement New feature or request

Comments

@TimoPtr
Copy link
Collaborator

TimoPtr commented Mar 20, 2025

Is your feature request related to a problem? Please describe.

In this comment #5132 (comment) we see that the naming was wrong. But Ktlint is not reporting this as an issue.

Describe the solution you'd like

Use Detekt that have rules for such things and have official plugin for gradle and Android Studio. It offer better experience also to extend than KTlint.
For some time we can keep both setup alongside they don't interfere with each other.

Additional context

I envision to add custom rules for our project like

  • forcing named params when there are multiple booleans without names (for better readability in PR)
  • forbid usage of android logger and encourage to use Timber
@TimoPtr TimoPtr added the enhancement New feature or request label Mar 20, 2025
@TimoPtr TimoPtr linked a pull request May 15, 2025 that will close this issue
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant