Regarding <https://github.com/arturbosch/detekt/issues/1143> is there a task that should run before ...

# detekt

[pavlospt]

Regarding https://github.com/arturbosch/detekt/issues/1143 is there a task that should run before executing detekt that would add Main class in classpath?

[mauin]

The detekt plugin should simply pass the extra formatting jar to the CLI. The only things required is that the detekt-cli jar is downloaded and that any extra jars are fetched. But that should be the case.

[mauin]

@pavlospt So you're looking into that issue? Then I'll pick up some other issue

[pavlospt]

Not sure I am capable of looking into it 😛 Since i am not so sure on the Gradle internals in this case

[pavlospt]

You can take it 🙂

[mauin]

Want me to take a look?

[pavlospt]

Sure 🙂

[mauin]

Okay. I'll see if I can figure something out.

[pavlospt]

Take a look at this one https://github.com/arturbosch/detekt/pull/1124 not sure if related!

[seb]

Yeah I’m sorry I haven’t had much time to look into that Gradle issue recently, it’s release week and everything is on fire all the time, as is often the case 😄

[mauin]

I'll submit a PR for the formatting jar issue in a minute and will then take a look at the report path issue.

[mauin]

I doubt that #1116 is really related to the detekt-formatting issue. I have a PR up for that one and that was really related to the dependency management of the Gradle Plugin

[seb]

Nah it’s a plugin classpath problem

[mauin]

Looks like 3flex already has a good understanding of the issue in #1116.

[seb]

Yeah I haven’t had a chance to do what he was asking, but it’s 100% reproducible by just checking out the branch

[seb]

I’m afraid I’m caught up on some P1 fixes on a release 😞

[mauin]

No worries. From how I understand it we should be able to fix the issue in another PR to unlock yours.

[seb]

brilliant, thanks!