You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@maven.apache.org by "elharo (via GitHub)" <gi...@apache.org> on 2026/07/23 15:03:40 UTC
[PR] Fix documentation issues in site pages [maven-shared-utils]
elharo opened a new pull request, #417:
URL: https://github.com/apache/maven-shared-utils/pull/417
Various documentation fixes:
- Upgrade plexus-utils link to HTTPS
- Reword "forked from" → "originated in" (more precise)
- Clarify ambiguous "the original" phrasing
- Remove redundant parenthetical "(different)"
- Use proper project reference "Commons versions"
- Use project display name "Maven Shared Utils"
- Upgrade XSD URL from HTTP to HTTPS in site.xml
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@maven.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Fix documentation issues in site pages [maven-shared-utils]
Posted by "elharo (via GitHub)" <gi...@apache.org>.
elharo merged PR #417:
URL: https://github.com/apache/maven-shared-utils/pull/417
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@maven.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Fix documentation issues in site pages [maven-shared-utils]
Posted by "Copilot (via GitHub)" <gi...@apache.org>.
Copilot commented on code in PR #417:
URL: https://github.com/apache/maven-shared-utils/pull/417#discussion_r3639229732
##########
src/site/apt/index.apt.vm:
##########
@@ -40,13 +40,13 @@ ${project.name}
Why?
- plexus-utils consisted mostly of code that was forked from various Apache projects.
- maven-shared-utils is based on the original from the Apache sources.
+ plexus-utils consisted mostly of code that originated in various Apache projects.
+ maven-shared-utils is based on those original Apache-source implementations.
Review Comment:
This sentence uses the artifactId-style name ("maven-shared-utils"), while later text in this section uses the display name ("Maven Shared Utils"). For consistency in the site docs, consider using the project display name here too (or reuse ${project.name}).
##########
src/site/apt/index.apt.vm:
##########
@@ -40,13 +40,13 @@ ${project.name}
Why?
- plexus-utils consisted mostly of code that was forked from various Apache projects.
- maven-shared-utils is based on the original from the Apache sources.
+ plexus-utils consisted mostly of code that originated in various Apache projects.
Review Comment:
Line has a trailing whitespace after the period. This can create noisy diffs in future edits and may violate whitespace rules if any are enforced.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@maven.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org