Skip to content

chore(ci): let java-parent pin the PomChecker version - #47

Merged
herbie-bot merged 1 commit into
mainfrom
chore/pomchecker-version
Sep 25, 2026
Merged

herbie-bot merged 1 commit into
mainfrom
chore/pomchecker-version

Conversation

@herbie-bot

Copy link
Copy Markdown
Collaborator

What

Drops the explicit 1.14.0 from the plugin coordinates in the Check POMs against the Maven Central rules step:

-  ./mvnw -B org.kordamp.maven:pomchecker-maven-plugin:1.14.0:check-maven-central
+  ./mvnw -B org.kordamp.maven:pomchecker-maven-plugin:check-maven-central

Why

java-parent already pins pomchecker-maven-plugin in its pluginManagement, at that same version. Naming it in the workflow too made it a second place to bump — and a place that could silently drift from the version the reactor otherwise uses. Maven resolves the version from pluginManagement when the goal is invoked without one.

Verification

Both runs against the current reactor:

  • Clean tree → exit 0, and the log confirms the resolved version: --- pomchecker:1.14.0:check-maven-central (default-cli) @ spring-services ---
  • <url> removed from spring-services-scim/pom.xml again → exit 1, with the same message that broke the 1.4.0 release: MavenCentral check failed: <url> is not defined in POM.

Identical behaviour to the pinned form.

🤖 Generated with Claude Code

The plugin coordinates in build.yml named 1.14.0 explicitly, which is the
version java-parent already pins in its pluginManagement — two places to
bump instead of one. Maven resolves the version from there when the goal
is invoked without it.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@herbie-bot
herbie-bot merged commit 453b40d into main Sep 25, 2026
1 check passed
@herbie-bot
herbie-bot deleted the chore/pomchecker-version branch September 25, 2026 06:53
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.

2 participants