Skip to content

Update oss-parent to 5, metricshub/workflows to v8 and the site to Doxia 2 - #59

Merged
NassimBtk merged 2 commits into
mainfrom
update/oss-parent-5-workflows-v7
Oct 9, 2026
Merged

NassimBtk merged 2 commits into
mainfrom
update/oss-parent-5-workflows-v7

Conversation

@NassimBtk

@NassimBtk NassimBtk commented Oct 8, 2026 •

Copy link
Copy Markdown
Contributor

GitHub runners now ship Maven 3.10.0 (image ubuntu24/20261004 and later). This PR aligns the project with MetricsHub/jflat and MetricsHub/simple-http-java.

What changed

Validation

  • jflat and simple-http-java use the same workflows (v8). Their deploys on Maven 3.10.0 end in BUILD SUCCESS, and new snapshots are published on Central (the origin step passes).
  • This PR's CI runs the build (and the site) on the runner's Maven. The Doxia 2 site already builds in CI in Issue #69 - Build the site with Doxia 2 jflat#70 and Issue #62 - Build the site with Doxia 2 simple-http-java#63.
  • After the merge, check the first Maven Deploy run on main:
    • no "Not using credentials of server 'central'" warning in the deploy step;
    • central-publishing:publish ends in BUILD SUCCESS;
    • a new snapshot timestamp appears on Central.

🤖 Generated with Claude Code

…xia 2

- pom.xml: parent org.metricshub:oss-parent 4 -> 5.
- .github/workflows: metricshub/workflows reusable workflows @V3 -> @v7
  (maven-build, maven-central-deploy, maven-central-release). GitHub runners
  now ship Maven 3.10, and Central snapshot deploys need workflows v7, which
  declares the central.sonatype.com origin.
- Site with Doxia 2: maven-site-plugin 4.0.0-M16 with
  org.sentrysoftware.maven:maven-skin-tools 1.8.01 (maven-site-plugin 3.12.1
  from oss-parent mixes Velocity 1.7 and 2.x and fails with Maven 3.10:
  AbstractMethodError on org.apache.velocity.tools.ToolContext), and
  src/site/site.xml converted to the SITE 2.0.0 descriptor with
  sentry-maven-skin 8.0.00, same as jflat (MetricsHub/jflat#70).

Co-Authored-By: Claude Opus 5.5 <[email protected]>
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 8, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-08T16:31:44.201663Z 9752ca5 PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

v8 keeps the Resolver local-repository files out of the Central bundle when releasing with Maven 3.10 (MetricsHub/workflows#39). The inputs are unchanged.

Co-Authored-By: Claude Opus 5.5 <[email protected]>
@NassimBtk NassimBtk changed the title Update oss-parent to 5, metricshub/workflows to v7 and the site to Doxia 2 Update oss-parent to 5, metricshub/workflows to v8 and the site to Doxia 2 Oct 8, 2026
@NassimBtk
NassimBtk merged commit 49fdad5 into main Oct 9, 2026
4 checks passed
@NassimBtk
NassimBtk deleted the update/oss-parent-5-workflows-v7 branch October 9, 2026 09:25
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