Point the source-tree links at maven-3.10.x rather than master - #434
Merged
Conversation
Four links on the Core ITs Suite pages addressed files by way of this
repository's master branch. Master was emptied to a four-file stub when the IT
tree moved into apache/maven ("Branch 3.10.x created"), so three of the four now
return 404 and the fourth reaches a signpost README rather than the instructions
the page promises. Each is repointed at maven-3.10.x, the branch this site is
built from, so the page and the tree it describes stay in step.
One of them was also addressing the wrong directory, independently of the branch:
bootstrap.txt is under src/test/resources-filtered, not src/test/resources, so
that link would have been dead on any ref.
The README link is repointed rather than left alone: master's README is now a
list of the surviving branches, while the steps for running the suite that the
sentence refers to are in the README on maven-3.10.x.
Every rewritten URL was fetched and returns 200 with no redirect. Verified by
building the core-it-suite site and comparing it with the build before this
change: the four link targets are the only difference.
Not addressed here, because it is release metadata rather than documentation and
it affects maven-3.9.x identically: <scm><tag>master</tag> in the POM makes the
generated "edit this page" links and the source-repository report point into the
same emptied master. Worth a committer's decision on both branches.
Generated-by: Claude Opus 5 (1M context)
slachiewicz
force-pushed
the
maven-3.10.x-site-links
branch
from
August 10, 2026 01:21
f5c83b9 to
633b584
Compare
slachiewicz
marked this pull request as ready for review
August 10, 2026 08:22
slawekjaranowski
approved these changes
Aug 10, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Four links on the Core ITs Suite pages address files by way of this repository's
masterbranch. Master was emptied to a four-file stub when the IT tree moved intoapache/maven(commit "Branch 3.10.x created"), so three of the four now return 404, and the fourth reaches a signpost README rather than the instructions the sentence promises.Base branch:
maven-3.10.x.Stacked on #433 — the pages only exist in Markdown form after that PR, so this branch is built on top of it and the diff shown here includes its two commits until it merges. Merge #433 first; this PR's own change is the single commit
Point the source-tree links at maven-3.10.x rather than master, two files, four lines.What changes
README.mdblob/master/README.md— 200, but master's README is now just a list of surviving branchesblob/maven-3.10.x/README.mdsettings.xmlblob/master/core-it-suite/src/test/resources-filtered/settings.xml— 404blob/maven-3.10.x/…bootstrap.txtblob/master/core-it-suite/src/test/**resources**/bootstrap.txt— 404blob/maven-3.10.x/core-it-suite/src/test/**resources-filtered**/bootstrap.txttree/master/core-it-suite/src/test/resources/bootstrap— 404tree/maven-3.10.x/…Two notes on the choices:
bootstrap.txtwas also addressing the wrong directory, independently of the branch — the file is undersrc/test/resources-filtered, notsrc/test/resources, so that link would have been dead on any ref. Both errors are fixed together.maven-3.10.xrather than a tag ormaster. The branch is what this site is built from, so the page and the tree it describes stay in step; a tag would freeze and drift. Master is what got us here, and it got there by being emptied rather than by being a branch. The bare repository-root link ("the official Core ITs Suite repository") is left alone — it resolves and is meant to name the repository, not a ref.Verification
curl -L: all four return 200,url_effectiveunchanged, so no redirect to a repository root. The three replaced ones were confirmed 404 first.mvn -N siteincore-it-suite, exit 0, compared against the build from Port the site documentation from APT to Markdown #433's head: the four link targets are the only difference in the generated pages.Not addressed here
<scm><tag>master</tag></scm>in the POM, with<url>…/tree/${project.scm.tag}</url>, points the generated "edit this page" link on every page and the source-repository report into the same emptied master. That is release metadata rather than documentation, andmaven-3.9.xcarries it identically, so it wants a committer's decision across both branches rather than a drive-by change in a docs PR.Drafted with Claude — please verify