Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Detekt deactivates non activated rules by default which causes confusion for detekt extension authors #1103

Closed
arturbosch opened this issue Sep 11, 2018 · 2 comments · Fixed by #4315
Closed

Detekt deactivates non activated rules by default which causes confusion for detekt extension authors #1103

arturbosch opened this issue Sep 11, 2018 · 2 comments · Fixed by #4315
Labels
api core improvement
Projects
Roadmap

Comments

@arturbosch
Copy link
Member

Expected Behavior

Rules from extensions - rule set id !in default rule set ids - should be active by default.

Current Behavior

Custom rules only executed if they are turned on in the yaml configuration.

Context

Questions from slack.
Any potential rule author.
@Mishkun
Copy link
Contributor

Mishkun commented Dec 24, 2021

Any news about this issue?

@BraisGabin
Copy link
Member

I think that #4315 should close this issue. The rule author will be able to decide if they rules are enabled or not by default after that.

@BraisGabin BraisGabin mentioned this issue Dec 24, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
api core improvement
Projects
Roadmap
  
Awaiting triage
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Allow plugins to contribute to the default configuration detekt/detekt
3 participants
@BraisGabin @Mishkun @arturbosch