GGraziadei opened a new issue, #8585: URL: https://github.com/apache/storm/issues/8585
The project is currently using a significantly outdated version of Checkstyle (v8.2). This version is several years old and lacks support for modern Java syntax and improved linting features found in the latest releases. https://mvnrepository.com/artifact/com.puppycrawl.tools/checkstyle/versions Continuing with this outdated version presents several issues: - Strict import enforcement: Newer versions of Checkstyle handle CustomImportOrder differently. We are currently seeing warnings (or being forced into specific import hacks) regarding the STATIC group because the old version's rules are inconsistent with modern IDE defaults. - Configuration inconsistency: the file `storm_checkstyle.xml` is based on an ancient version of Google’s checkstyle config (v7.7). Many properties used in our XML are deprecated or have been removed in Checkstyle 10+, making the current configuration incompatible with the latest tooling. - Modern Java support. Proposed Changes: - Bump dependency - Update `storm_checkstyle.xml` -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
