Regarding <https://github.com/arturbosch/detekt/is...
# detekt
p
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?
m
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.
@pavlospt So you're looking into that issue? Then I'll pick up some other issue
p
Not sure I am capable of looking into it 😛 Since i am not so sure on the Gradle internals in this case
You can take it 🙂
m
Want me to take a look?
p
Sure 🙂
m
Okay. I'll see if I can figure something out.
🎉 1
p
Take a look at this one https://github.com/arturbosch/detekt/pull/1124 not sure if related!
s
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 😄
m
I'll submit a PR for the formatting jar issue in a minute and will then take a look at the report path issue.
👍 2
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
👍 1
s
Nah it’s a plugin classpath problem
m
Looks like 3flex already has a good understanding of the issue in #1116.
s
Yeah I haven’t had a chance to do what he was asking, but it’s 100% reproducible by just checking out the branch
I’m afraid I’m caught up on some P1 fixes on a release 😞
m
No worries. From how I understand it we should be able to fix the issue in another PR to unlock yours.
s
brilliant, thanks!