Skip to content

docs: fix broken reference links#11692

Open
amahi1568 wants to merge 1 commit into
apache:masterfrom
amahi1568:docs-fix-reference-links
Open

docs: fix broken reference links#11692
amahi1568 wants to merge 1 commit into
apache:masterfrom
amahi1568:docs-fix-reference-links

Conversation

@amahi1568

Copy link
Copy Markdown

Description

This pull request addresses a single documentation-related issue by fixing a minor typo / improving clarity in the existing documentation.

There are no functional or behavioral code changes included in this PR.
The change is limited strictly to documentation and does not impact runtime behavior.

Checklist

  • Your pull request should address just one issue, without pulling in other changes.
  • Write unit tests that match behavioral changes (not applicable for documentation-only changes).
  • Run mvn verify (not required for documentation-only changes).
  • You have run the Core IT (not applicable).

License Declaration

  • I hereby declare this contribution to be licenced under the Apache License Version 2.0, January 2004

Comment thread pom.xml
<distributionShortName>Maven</distributionShortName>
<distributionName>Apache Maven</distributionName>
<maven.site.path>ref/4-LATEST</maven.site.path>
<maven.site.path>ref/4.0.0</maven.site.path>

@slawekjaranowski slawekjaranowski Feb 12, 2026

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

on master branch it is ok, we should change it on maven-4.0.x branch to something like ref/4.0-LATEST
@gnodet, @cstamas what do you think?

is is used to publish site candidate during release
https://maven.apache.org/components/ref/

maybe we should also add redirects to:
https://github.com/apache/maven-site/blob/master/content/filtered-resources/.htaccess

@gnodet, @cstamas what do you think?

@gnodet gnodet 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.

Claude Code on behalf of Guillaume Nodet

This change would actually break the site path rather than fix it.

The master branch is currently developing 4.1.0-SNAPSHOT. The 4-LATEST path is intentionally used so the generated site always points to the latest 4.x reference docs, regardless of the specific release version. Hardcoding 4.0.0 here would be incorrect.

Comment thread pom.xml
<distributionShortName>Maven</distributionShortName>
<distributionName>Apache Maven</distributionName>
<maven.site.path>ref/4-LATEST</maven.site.path>
<maven.site.path>ref/4.0.0</maven.site.path>

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.

This change should not be made. 4-LATEST is the correct value — it's an intentional alias that always resolves to the latest 4.x release docs on the Maven site.

You can verify:

Additionally, master is currently on 4.1.0-SNAPSHOT, so hardcoding 4.0.0 would be doubly wrong.

@gnodet gnodet 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.

This change would break the site links — ref/4-LATEST resolves correctly (HTTP 200) while ref/4.0.0 returns 404. The 4-LATEST path is intentionally used so generated docs always point to the latest 4.x reference, regardless of release version.

Claude Code on behalf of Guillaume Nodet

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.

4 participants