Roman Ivanov

Update verify-no-exception-configs validation

3

only maintainers can address this issue. Make validation to use secure variable to avoid security concern. ``` # Ensure that all modules are used in no exception configs - env:...

Extend suppression configuration to be more fine grained

18

When new Static Analysis Tool (SAT) is introduced it always find a lot of violations in code. But it is not always possible to easily fix them (number of reasons)....

upgrade to latest IDEA inspection engine after issue in IDEA engine is fixed

4

as https://youtrack.jetbrains.com/issue/IDEA-293720 fixed, try to upgrade to latest IDEA inspection engine for now we downgraded to 2020.1.1 https://teamcity-support.jetbrains.com/hc/en-us/requests/4153345

Migrate all jobs from Wercker to Travis or any other CI

2

Wercker will be shut down on oct 31 2022 we need to stop using it. Move all to Travis or any other CI, main repo has bunch CI in use.

import should work with rule templates

27

With SonarQube 4.5.1 and Checkstyle plugin 2.2, importing the following file triggers an error: "Rule template can't be activated on a Quality profile: checkstyle:com.puppycrawl.tools.checkstyle.checks.regexp.RegexpSinglelineJavaCheck" ``` ```

Sonar 4.36 (based on checkstyle 8.36) All descriptions of modules become a bit unpleasant to read due to javadoc tags (example `{@code allowedAbbreviationLength}` `{@code ignoreFinal}, {@code ignoreStatic}, and {@code ignoreStaticFinal}...

Automate release process

1

Right now release process is a bit manual https://github.com/checkstyle/sonar-checkstyle/wiki/How-to-make-release I already did automation of github related steps: https://github.com/checkstyle/eclipse-cs/blob/master/release.sh#L89 same will be done soon at main repo. TODO: make same release.sh...

get rid of dependency to sonar-java-plugin

2

this dependency is required only do smth with classloader. We should implement this ourself. We should not depend on this artifact at all.

Resolve violations from sonarcloud.io

8

https://sonarcloud.io/dashboard?id=checkstyle_sonar-checkstyle lets try to resolve all violations in our code, if reasonable.

http://localhost:9000/coding_rules/#q=Import%20Order no way to remove STATIC_IMPORT token, it is default. https://github.com/checkstyle/checkstyle/blob/master/src/main/java/com/puppycrawl/tools/checkstyle/checks/imports/ImportOrderCheck.java#L348 http://checkstyle.sourceforge.net/config_imports.html#ImportOrder fyi , required tokens are appended automatically: https://github.com/checkstyle/checkstyle/blob/master/src/main/java/com/puppycrawl/tools/checkstyle/TreeWalker.java#L211