rnveach

Cloning the fork or the main repo has the same effect. This is a windows depth issue and the folders are too deep in the repo.

Yes.

> clean Meant maven clean. > from Eclipse ? what version of eclipse and jacoco ? Jacoco is the version checkstyle uses in master. Eclipse is Luna SR2 and Oxygen...

Linux maven does the same thing. I can reproduce the issue with just `mvn clean test` without verify and examining the generated class files for instances of `jacoco` text. @romani...

> which is right after the phase test. As I stated, I can see jacoco in the class files **before** the test phase is complete. It doesn't need to wait...

> Perhaps this is due to the update of the JaCoCo to version 0.8.1 I rolled back to 7be8266142e1f719d82ed2b83a69f30cbed8cb33 (right before 8.1) and still see the issue.

> We need to try the recommended way. If you know how, give it a try. I'm not seeing it yet. Removing `default-instrument` and `default-restore-instrumented-classes` fixed the classes but jacoco...

Since we can't disable offline mode, we should probably update contributing docs to notify users on this issue if they do full `verify`. They will need to split `verify` between...

@romani Yes. We are still forced to do split runs between jacoco and non-jacoco.

Nothing has been done to mitigate it, so it is still an issue.