rnveach
rnveach
@romani The examples in the issue do not compile and can't be copied to PR as it is. ```` > javac .\Test.java .\Test.java:8: error: method foo() is already defined in...
@romani I see it as pointless to continue this PR. My prior denial and block message has not changed. Even the fixes I see you doing now don't fix the...
Code has not changed, so my review hasn't changed and this should still be blocked. Even admitting we will revert most of this PR shows this isn't a good step...
Sorry for the delay as I was at my job around the clock last week. > Edit: Just tried with v1.3.3 and am able to reproduce your issue. So it...
@Bananeweizen @Calixte Is there something stopping us finish the releases before doing more PRs, so we can atleast be at the same version as CS? I was hoping for eclipse-cs...
sevntu and eclipsecs-sevntu are seen as one unit. https://github.com/sevntu-checkstyle/sevntu.checkstyle/blob/master/sevntu-checks/pom.xml#L18-L19 We cannot upgrade one without breaking the other, and it would double the release load for the small repo to release...
@hcoles Checkstyle has not been able to upgrade to pitest 1.4.10 because of https://github.com/hcoles/pitest/issues/673 . And this issue details why we cannot upgrade to 0.11 for pitest-junit5 . You can...
> This will only work for fall throughs. Adding the redundant break in the code below would result in jumps to different locations and prevent the filtering from happening. Yes,...
@hcoles Thanks for releasing the newest pitest. We are doing an update at https://github.com/checkstyle/checkstyle/pull/12707 and it is resolving mutations that we expected. The following is not be resolved. ```` case...
@martin-mfg This PR will need changes similar to https://github.com/checkstyle/checkstyle/pull/7418 for the CI. This wercker is failing on `NoErrorTest - Spring Cloud GCP`. @romani I recommend we merge the others first...