Skip to content

chore: import ordering, final parameters, dependency analysis, and update deps#30

Merged
deer merged 1 commit intomainfrom
consistency
Apr 2, 2026
Merged

chore: import ordering, final parameters, dependency analysis, and update deps#30
deer merged 1 commit intomainfrom
consistency

Conversation

@deer
Copy link
Copy Markdown
Collaborator

@deer deer commented Apr 2, 2026

A maintenance pass enforcing build hygiene and code style consistency across all 23 modules.

  • New build enforcement: Added maven-dependency-plugin (analyze-only, failOnWarning=true) to enforce explicitly declared dependencies; added requireMavenVersion enforcer rule (3.9.14); pinned Checkstyle to 13.3.0 via explicit plugin dependency.
  • New Checkstyle rules: CustomImportOrder (third-party → standard Java → static, alphabetically sorted with blank line separators), FinalParameters, UnusedImportsCheck. Removed stale commented-out Workday-specific checks.
  • Dependency bumps: jackson-core 2.21.0 → 2.21.2, JUnit 6.0.1 → 6.0.3, Mockito 5.18.0 → 5.23.0, Maven wrapper 3.9.12 → 3.9.14.
  • Code conformance: Import blocks reordered and parameters made final throughout all source and test files. Module pom.xml files trimmed of now-redundant transitive dependency declarations.

Test plan:

  • ./mvnw clean install passes with no dependency warnings and no Checkstyle violations

@deer deer merged commit 461cd56 into main Apr 2, 2026
3 checks passed
@deer deer deleted the consistency branch April 2, 2026 09:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant