Hi guys, An update regarding the detekt intellij ...

# detekt

[Dmytro Primshyts]

Hi guys,

An update regarding the detekt intellij plugin:

* I've added the project configuration (tools -> Detekt) with following controls (can't paste a screenshot here)
** Checkbox "Enable Detekt" (checked by default): determines whether or not detekt should be used for current project
** Checkbox "Analyze Test Code" (unchecked by default): determines whether or not detekt should analyze test sources
** Selection for Detekt version (just a dummy placeholder for by now)
** Configuration file: path to detekt rules

Few things to discuss:

1. In case if a rule throws an error the whole analysis fails

I think that detekt should provide some mechanism to mark such rules as erroneous and write the errors into some log (detekt allows custom rules to be added, how those should be debugged?)

could be enabled by some flag (--skip-errored-rules?)

2. If a project uses detekt within the build process, the plugin should allow to use the same version of detekt

Any ideas how we could achieve that?
I suppose, simply loading corresponding detekt-cli from the repo would do the thing

3. The plugin should be able to pickup detekt rules file (and version of detekt) from a project sources (manually setting the rules file is bad UX)

In order to achieve that the rules file should have some marker that will allow the plugin to determine for sure that the file is the detekt file (I suppose detektVersion: xxxx will do)