Copilot commented on code in PR #12638:
URL: https://github.com/apache/maven/pull/12638#discussion_r3684844157


##########
src/site/markdown/configuring.md:
##########
@@ -90,13 +90,21 @@ from `${session.rootDirectory}/.mvn/maven-user.properties`.
 
 ## Settings
 
+Settings are configured in the `~/.m2/settings.xml` file (user level) 
+and `${maven.installation.conf}/settings.xml` (Maven installation level). See 
+[Settings Reference](./api/maven-api-settings/settings.html) for details.
+
 ## Extensions
 
+Extensions allow you to add additional functionality to Maven. They are 
+configured in the `~/.m2/extensions.xml` file (user level) and 
+`${maven.installation.conf}/extensions.xml` (Maven installation level).

Review Comment:
   The Extensions section mentions only user- and installation-level 
`extensions.xml`, but Maven also supports project-level extensions via 
`${maven.project.conf}/extensions.xml` (`maven.project.extensions`). The 
documentation should include the project-level location for completeness.



##########
src/site/markdown/configuring.md:
##########
@@ -90,13 +90,21 @@ from `${session.rootDirectory}/.mvn/maven-user.properties`.
 
 ## Settings
 
+Settings are configured in the `~/.m2/settings.xml` file (user level) 
+and `${maven.installation.conf}/settings.xml` (Maven installation level). See 
+[Settings Reference](./api/maven-api-settings/settings.html) for details.

Review Comment:
   The Settings section describes user- and installation-level `settings.xml`, 
but Maven also supports a project-level settings file (see 
`maven.project.settings` / `${maven.project.conf}/settings.xml`). Omitting it 
can mislead users about available configuration levels.



##########
src/site/markdown/cache-configuration.md:
##########
@@ -18,7 +18,7 @@ under the License.
 -->
 # Maven Cache Configuration Enhancement
 
-This document describes the enhanced cache configuration functionality in 
Maven's DefaultRequestCache.
+This document describes the enhanced cache configuration functionality in 
DefaultRequestCache.

Review Comment:
   For consistency with other type/class references in this document (which use 
code formatting), `DefaultRequestCache` should be wrapped in backticks so it's 
clearly a class name.



-- 
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]

Reply via email to