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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]