Skip to content

Check Style Cleanup#398

Merged
kevin-mcgoldrick merged 1 commit intomasterfrom
CheckStyleCleanup
May 23, 2025
Merged

Check Style Cleanup#398
kevin-mcgoldrick merged 1 commit intomasterfrom
CheckStyleCleanup

Conversation

@kevin-mcgoldrick
Copy link
Collaborator

@kevin-mcgoldrick kevin-mcgoldrick commented May 16, 2025

Fix CheckStyle plugin usage

  • Update checkstyle version
  • Update checkstyle configuration file
  • Update static version to project.version in jacoco-report-aggregator
  • Consistent xml boilerplate

Please make sure these check boxes are checked before submitting

  • ** Squashed Commits **
  • ** All Tests Passed ** - mvn clean test -P default

** PR review process **

  • Requires one +1 from a reviewer
  • Repository owners will merge your PR once it is approved.

@kevin-mcgoldrick kevin-mcgoldrick merged commit 76b545b into master May 23, 2025
3 checks passed
@kevin-mcgoldrick kevin-mcgoldrick deleted the CheckStyleCleanup branch May 23, 2025 17:41
@kevin-mcgoldrick kevin-mcgoldrick restored the CheckStyleCleanup branch July 15, 2025 16:07
@kevin-mcgoldrick kevin-mcgoldrick deleted the CheckStyleCleanup branch July 15, 2025 16:30
@kevin-mcgoldrick kevin-mcgoldrick restored the CheckStyleCleanup branch July 15, 2025 18:12
@kevin-mcgoldrick kevin-mcgoldrick deleted the CheckStyleCleanup branch July 15, 2025 18:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants