rnveach
rnveach
> It does seem like it could be problematic to pull values from the config we are testing. PR #11652 is blocked because it will be reworked to what I...
https://checkstyle.org/releasenotes_old_6-0_7-8.html#Release_7.4 https://checkstyle.org/releasenotes_old_6-0_7-8.html#Release_7.5 https://checkstyle.org/releasenotes_old_6-0_7-8.html#Release_7.5.1 > Checkstyle moving to the Worker API What is Worker API? Can you give an example? Edit: If Worker API is https://docs.gradle.org/current/userguide/worker_api.html , then this is not...
From what I have looked at and understand thus far. The issue is the defined suppressions are not working when run through Gradle plugin. It looks like gradle plugin is...
Since this is not being done on windows, this will most likely mean the following issue , https://github.com/checkstyle/checkstyle/issues/8884 , will have to be fixed for this issue to be completed...
http://rveach.no-ip.org/checkstyle/regression/pitest-reports/38/pitest-blocks/com.puppycrawl.tools.checkstyle.checks.blocks/EmptyBlockCheck.java.html#org.pitest.mutationtest.report.html.SourceFile@3ceb8e1f_203 Shows we are sending a variable to the `log` method and not even making use of it in the message display.
I am fine with the new util method.
> https://checkstyle.sourceforge.io/version/6.18 - is good link @romani Why doesn't https://checkstyle.org/version/6.18/ work? Why don't we move everything to checkstyle.org as that is our site and we should move everyone off of...
Having this disabled made us forget to merge https://github.com/checkstyle/contribution/pull/548 from over a year ago. I asked @nick-mancuso to run regression with that PR to ensure we won't be hit when...
@romani > use secure variable to avoid security concern What is security concern? If private, share details in PM. === Trying to run failed. ```` $ ./.ci/validation.sh verify-no-exception-configs .ci-temp/verify-no-exception-configs/checks-nonjavadoc-error.xml:4.63: failed...
Running test in https://github.com/checkstyle/checkstyle/pull/12300 and it seems the verify works despite the failure messages. Sevntu's CI has the same as well.