gnodet commented on code in PR #1597:
URL: https://github.com/apache/maven/pull/1597#discussion_r1671316856


##########
apache-maven/src/assembly/maven/conf/settings.xml:
##########
@@ -29,14 +29,14 @@ under the License.
  |
  |                 -s /path/to/user/settings.xml
  |
- |  2. Global Level. This settings.xml file provides configuration for all 
Maven
+ |  2. System Level. This settings.xml file provides configuration for all 
Maven

Review Comment:
   One problem is that project settings are validated differently than user or 
install settings...  If we have a single list, it will be hard to keep that 
information.  I think this is the only case where those xml files (toolchains, 
settings, extensions) are loaded differently.
   
   Another problem which is not currently addressed, is the merge of 
`extensions.xml`, but that's a different topic...



-- 
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-unsubscr...@maven.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to