Skip to content

chore(deps): bump basepom to 65.1 - #1361

Closed
jasmith-hs wants to merge 2 commits into
master-2.7.xfrom
jasmith_hubspot/basepom-65.1-2.7.x
Closed

jasmith-hs wants to merge 2 commits into
master-2.7.xfrom
jasmith_hubspot/basepom-65.1-2.7.x

Conversation

@jasmith-hs

Copy link
Copy Markdown
Contributor

Description

Bumps the basepom parent POM from 63.4 to 65.1 on the master-2.7.x branch.

BRAVE

Backwards Compatibility

  • Parent POM version bump only; no application code changes
  • Full mvn clean verify passes locally (2040 tests, 0 failures)

Rollout and Rollback Plan

  • Standard release process for a 2.7.x patch
  • Revert this commit to roll back if a plugin/version change in basepom 65.1 causes issues

Automated Testing

  • Relies on existing unit/integration test suite plus static analysis checks (SpotBugs, PMD, duplicate-finder) that run as part of the build

Verification

  • Ran mvn clean verify locally against this branch; build succeeded with no new warnings beyond pre-existing ones

Expect Dependencies to Fail

  • None expected; this only updates build tooling versions, not runtime dependencies
REVIEWERS: Please review both the code changes and the answers above, and validate that they match the expectations for BRAVE

This PR description was authored by Claude Code.

🤖 Generated with Claude Code

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@jasmith-hs jasmith-hs self-assigned this Oct 29, 2026
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@jasmith-hs

Copy link
Copy Markdown
Contributor Author

Closing in favor of #1362.

The original motivation for this basepom bump was to fix a Central-publish failure on the oss-release-jinjava-2-7-x-release cronjob caused by central-publishing-maven-plugin 0.7.0 choking on a new "warnings" field in Sonatype's deployment-status response. Bumping all the way to basepom 65.1 does pull in a fixed plugin version (0.9.0), but it also drags in unrelated changes — notably basepom.build.phase-toolchain flipping from never to validate, which broke the public GitHub Actions jdk-11/jdk-17 CI matrix (toolchain + checkstyle-plugin JDK requirements).

basepom exposes the plugin version as an overridable property, so #1362 sets dep.plugin.central-publishing.version to 0.9.0 directly without bumping the basepom parent, avoiding all of that unrelated fallout.

This comment and PR closure were authored by Claude Code.

@jasmith-hs jasmith-hs closed this Oct 29, 2026
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