Skip to content
This repository has been archived by the owner on Feb 11, 2022. It is now read-only.

Add per-tool evaluation #164

Closed
georgi-neykov-hub opened this issue Dec 20, 2018 · 2 comments
Closed

Add per-tool evaluation #164

georgi-neykov-hub opened this issue Dec 20, 2018 · 2 comments

Comments

@georgi-neykov-hub
Copy link

First of all a big thank you for the effort on the plugin and making and making it open source.

Are you considering adding the option to have an violation collection and evaluation tasks for each tool? It would be of great convenience and it would allow running tasks for the different tools in parallel in a CI environment. For example, in v0.8.0 of the plugin there is no way to evaluate only the errors for Checkstyle, I would prefer stopping a CI build from running unit tests, lint, ... and other heavy tasks if there style problems with the source files .

@tasomaniac
Copy link
Contributor

Hi, if you have Gradle parallel builds available, running evaluateViolations should run all other tools in parallel already.

I created this issue a while ago for fail fast option. I think this captures what you want to do.
#112

I put some thoughts onto this and it is not super easy. It is easy to make individual tools fail the build as early as possible, but then it is not so easy to have consistent reports at the end.

@mr-archano
Copy link
Contributor

Hi @georgi-neykov-hub we've been discussing about having per-tool reporting in the past and as @tasomaniac we haven't found a strategy that makes us happy yet.

I believe that we could tackle this in separate steps, starting for instance with addressing #43 (Use file output to store violations per tool), combined with a custom evaluator (introduced in #45 and documented in docs/incubating/custom-evaluator.md).

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants