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.

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

Custom rules integration to IntelliJ plugin? #43

Closed
mtiidla opened this issue Sep 3, 2019 · 1 comment · Fixed by #54
Closed

Custom rules integration to IntelliJ plugin? #43

mtiidla opened this issue Sep 3, 2019 · 1 comment · Fixed by #54
Labels
feature New feature or request good first issue Good for newcomers
Milestone

Comments

@mtiidla
Copy link

mtiidla commented Sep 3, 2019

I have created a custom detekt rule and integrated it into the CI process. I am also using the IntelliJ plugin for Android Studio which works like a charm highlighting the issues based on the detekt.yml configuration file.

I have tried to add my ruleset to that file with the following config:

my_project_detekt_rules:
  NoKoinExplicitReturnType:
    active: true

I have also selected the .yml file in the IDE plugin as the configuration file and can confirm it registers changes in the yml file (IDE won't show warnings on a issue if I disable it in the .yml file)

However, the IDE does not seem to pick up my custom rule to highlight the issues, is that even supported by the plugin, or how should I approach that?

Using version 1.0.1

First opened in detekt/detekt#1864

@arturbosch arturbosch added feature New feature or request good first issue Good for newcomers labels Sep 6, 2019
@arturbosch
Copy link
Member

Thanks for the feedback.
Yes, an input field is missing in the config which basically will delegate to --plugins.

@arturbosch arturbosch added this to the 0.4.0 milestone Dec 16, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature or request good first issue Good for newcomers
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants