Skip to content

docs: document plugin.xml V4 vs V3 - #13261

Merged
gnodet merged 1 commit into
apache:maven-3.10.xfrom
gnodet:backport/13253-to-3.10.x
Sep 24, 2026
Merged

gnodet merged 1 commit into
apache:maven-3.10.xfrom
gnodet:backport/13253-to-3.10.x

Conversation

@gnodet

@gnodet gnodet commented Sep 24, 2026

Copy link
Copy Markdown
Contributor

Backport of #13253 (4398d65) to maven-3.10.x.

Conflict resolution:

  • maven-plugin-api/src/main/mdo/plugin.mdo: descriptions from the commit applied, version tags kept at 1.0.0 (not 1.0.0+), V4-only resolutions field discarded
  • maven-model-builder/src/site/markdown/super-pom.md.vm: restored (false conflict from git path mapping)
  • compat/maven-plugin-api/src/main/mdo/plugin.mdo: removed (path does not exist on 3.10.x flat layout)

@gnodet gnodet added documentation Improvements or additions to documentation backport labels Sep 24, 2026
@gnodet gnodet added this to the 3.10.0 milestone Sep 24, 2026

@gnodet-bot gnodet-bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Docs backport is faithful and accurate. One minor nit in plugin.mdo.

This review was generated by an AI agent, Hermès on behalf of @gnodet.

Comment thread maven-plugin-api/src/main/mdo/plugin.mdo Outdated

@gnodet-bot gnodet-bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Previous nit addressed — plugin.mdo now has the trailing newline. The rest of the backport is faithful: site.xml nav cleanup, .mdo description additions, and the index.md updates all match the source PR #13253 accurately. The lifecycle-mappings.html menu removal is consistent with the parent commit. Clean.

This review was generated by an AI agent, Hermès on behalf of @gnodet.

@gnodet
gnodet force-pushed the backport/13253-to-3.10.x branch 2 times, most recently from 5f3e0a3 to b326659 Compare September 24, 2026 08:30
Backport of apache#13253 to maven-3.10.x.

Fills in previously empty descriptions for the following fields
in maven-plugin-api/src/main/mdo/plugin.mdo:
- goalPrefix, isolatedRealm, inheritedByDefault (PluginDescriptor)
- executeLifecycle, composer, parameters (MojoDescriptor)
- roleHint (Requirement)
@gnodet
gnodet force-pushed the backport/13253-to-3.10.x branch from b326659 to 7a05c5e Compare September 24, 2026 08:31

@gnodet-bot gnodet-bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Previous nit (trailing newline on plugin.mdo) has been addressed in this squash. All seven filled descriptions are accurate and consistent with the source PR #13253:

  • goalPrefix, isolatedRealm, inheritedByDefault on PluginDescriptor ✓
  • executeLifecycle, composer, parameters on MojoDescriptor ✓
  • roleHint on Requirement ✓

The three remaining empty <description> tags (configuration, and both implementation fields) are pre-existing on maven-3.10.x and outside this PR's stated scope — not a concern here.

Clean backport.

This review was generated by an AI agent, Hermès on behalf of @gnodet.

@gnodet-bot gnodet-bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Nit from prior review (trailing newline) addressed via squash. Descriptions are accurate, faithful to V3 semantics, and consistent with the source PR #13253. Clean.

This review was generated by an AI agent, Hermès on behalf of @gnodet.

@gnodet
gnodet merged commit 82a0f20 into apache:maven-3.10.x Sep 24, 2026
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants