Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[improve][misc] Upgrade checkstyle to 10.14.2 #22291

Merged
merged 1 commit into from
Mar 18, 2024

Conversation

nodece
Copy link
Member

@nodece nodece commented Mar 18, 2024

Fixes #22286

Motivation

Upgrade checkstyle to 10.14.2 for support IntelliJ checkstyle plugin@5.88.

Modifications

  • Upgrade checkstyle from 8.37 to 10.14.2

Documentation

  • doc
  • doc-required
  • doc-not-needed
  • doc-complete

@nodece nodece self-assigned this Mar 18, 2024
@nodece nodece added this to the 3.3.0 milestone Mar 18, 2024
@github-actions github-actions bot added the doc-not-needed Your PR changes do not impact docs label Mar 18, 2024
@nodece
Copy link
Member Author

nodece commented Mar 18, 2024

/pulsarbot rerun-failure-checks

@nodece nodece force-pushed the upgrade-checkstyle branch from 0221b5d to 02562d6 Compare March 18, 2024 11:32
@nodece nodece requested a review from lhotari March 18, 2024 12:17
@nodece
Copy link
Member Author

nodece commented Mar 18, 2024

/pulsarbot rerun-failure-checks

Copy link
Member

@lhotari lhotari left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Good work @nodece!

@lhotari lhotari merged commit cd512e4 into apache:master Mar 18, 2024
50 of 51 checks passed
lhotari pushed a commit that referenced this pull request Mar 18, 2024
(cherry picked from commit cd512e4)

# Conflicts:
#	pom.xml
lhotari pushed a commit that referenced this pull request Mar 18, 2024
lhotari pushed a commit that referenced this pull request Mar 18, 2024
nikhil-ctds pushed a commit to datastax/pulsar that referenced this pull request Apr 3, 2024
(cherry picked from commit cd512e4)
(cherry picked from commit 31c5206)
nikhil-ctds pushed a commit to datastax/pulsar that referenced this pull request Apr 4, 2024
(cherry picked from commit cd512e4)
(cherry picked from commit 31c5206)
hanmz pushed a commit to hanmz/pulsar that referenced this pull request Feb 12, 2025
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

IntelliJ checkstyle plugin dropped support for checkstyle 8.x, upgrade checkstyle to newer version
2 participants