Fix documentation issues in site pages#417
Merged
Merged
Conversation
- Upgrade plexus-utils link to HTTPS - Reword 'forked from' -> 'originated in' - Clarify ambiguous 'the original' phrasing - Remove redundant parenthetical (different) - Use proper project name 'Commons versions' - Use project display name 'Maven Shared Utils' - Upgrade XSD URL from HTTP to HTTPS in site.xml
Contributor
There was a problem hiding this comment.
Pull request overview
This PR updates the Maven Shared Utils project site documentation to improve clarity and consistency, and to modernize external references by switching relevant URLs to HTTPS.
Changes:
- Switch the plexus-utils link in the site index page from HTTP to HTTPS.
- Reword several sentences in the site index page for clearer, more precise phrasing and naming.
- Switch the Maven site decoration XSD URL in
site.xmlfrom HTTP to HTTPS.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| src/site/site.xml | Updates the decoration XSD schema URL to HTTPS. |
| src/site/apt/index.apt.vm | Updates external link to HTTPS and refines wording/naming in the intro content. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
ascheman
approved these changes
Jul 23, 2026
ascheman
left a comment
ascheman
left a comment
There was a problem hiding this comment.
Thanks Elliott — nice cleanup. I checked each change:
- HTTPS upgrades (plexus-utils link in
index.apt.vm, XSDschemaLocationinsite.xml) — correct. Good catch leaving thexmlns=decoration namespace onhttp://; that's an identifier, not a URL, so it must stay as-is. - Wording — "originated in" over "forked from", dropping the redundant "(different)", and "Commons versions" all read more precisely.
- Naming — standardizing the prose on "Maven Shared Utils" (the
${project.name}display form) is the right call; both remaining prose mentions are now consistent.
Copilot's consistency note is already satisfied by the final state of the file — no lowercase maven-shared-utils left in prose. CI is green across all 11 jobs. LGTM. 👍
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.
Various documentation fixes: