You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pulsar.apache.org by "lhotari (via GitHub)" <gi...@apache.org> on 2026/03/31 11:56:19 UTC

[PR] [improve][build] Upgrade Checkstyle from 10.14.2 to 13.3.0 [pulsar]

lhotari opened a new pull request, #25440:
URL: https://github.com/apache/pulsar/pull/25440

   ### Motivation
   
   Upgrade Checkstyle from 10.14.2 to 13.3.0 to get the latest rule improvements, bug fixes, and Java 21 support. The old DTD URLs pointing to `puppycrawl.com` are deprecated and have been updated to `checkstyle.org`.
   
   ### Modifications
   
   - Updated Checkstyle version from `10.14.2` to `13.3.0` in `gradle/libs.versions.toml`
   - Updated DTD URL in `buildtools/src/main/resources/pulsar/checkstyle.xml` from `http://www.puppycrawl.com/dtds/configuration_1_3.dtd` to `https://checkstyle.org/dtds/configuration_1_3.dtd`
   - Updated DTD URL in `buildtools/src/main/resources/pulsar/suppressions.xml` from `http://www.puppycrawl.com/dtds/suppressions_1_1.dtd` to `https://checkstyle.org/dtds/suppressions_1_1.dtd`
   
   ### Verifying this change
   
   - [x] Make sure that the change passes the CI checks.
   
   This change is a trivial rework / code cleanup without any test coverage.
   
   Verified locally with `./gradlew checkstyleMain checkstyleTest` — all 303 tasks passed successfully.
   
   ### Does this pull request potentially affect one of the following parts:
   
   - [x] Dependencies (add or upgrade a dependency)
   - [ ] The public API
   - [ ] The schema
   - [ ] The default values of configurations
   - [ ] The threading model
   - [ ] The binary protocol
   - [ ] The REST endpoints
   - [ ] The admin CLI options
   - [ ] The metrics
   - [ ] Anything that affects deployment
   
   ### Documentation
   
   - [x] `doc-not-needed`
   
   ### Matching PR in forked repository
   
   PR in forked repository: <!-- ENTER URL HERE -->


-- 
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: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] [improve][build] Upgrade Checkstyle from 10.14.2 to 13.3.0 [pulsar]

Posted by "lhotari (via GitHub)" <gi...@apache.org>.
lhotari merged PR #25440:
URL: https://github.com/apache/pulsar/pull/25440


-- 
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: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org