Skip to content

Update extra-enforcer-rules to 1.12.1 and drop its maven-dependency-tree pin - #18796

Merged
HTHou merged 1 commit into
apache:masterfrom
slachiewicz:drop-maven-dependency-tree
Oct 8, 2026
Merged

HTHou merged 1 commit into
apache:masterfrom
slachiewicz:drop-maven-dependency-tree

Conversation

@slachiewicz

Copy link
Copy Markdown
Member

Description

Update extra-enforcer-rules and drop the maven-dependency-tree pin

extra-enforcer-rules 1.7.0 needed maven-dependency-tree on the enforcer classpath, which is why it is pinned next to it. Releases from 1.8.0 on no longer use it, so this updates extra-enforcer-rules to 1.12.1, the current release, and removes the pin.

Verified: mvn -N -P enforce validate → all five enforcer executions pass, including BanDuplicateClasses and the OSS Index check; mvn -N spotless:check passes.


This PR has:

  • been self-reviewed.

Key changed/added classes (or packages if there are too many classes) in this PR

Root pom.xml only.

…ree pin

extra-enforcer-rules 1.7.0 needed maven-dependency-tree on the enforcer classpath; 1.8.0 and later do not use it.
@HTHou
HTHou requested a balanced review from Copilot October 8, 2026 02:51

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟢 Approval recommended

The new version exists, removes the pinned dependency requirement, and introduces no unresolved issues.

0 open findings

What changed in this PR

Updates the root Maven Enforcer configuration to use the latest stable Extra Enforcer Rules release.

Changes:

  • Upgrades extra-enforcer-rules from 1.7.0 to 1.12.1.
  • Removes the obsolete maven-dependency-tree classpath pin.
File Description
pom.xml Updates and simplifies Maven Enforcer dependencies.

🧠 Review effort: Balanced


Give feedback about Copilot approvals in this survey to enter a drawing for a $150 gift card.

@HTHou
HTHou merged commit 28c1938 into apache:master Oct 8, 2026
39 checks passed
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.

3 participants