[GitHub] [maven] gnodet commented on a diff in pull request #1185: [MNG-7830] Switch from plexus-xml to stax / woodstox

2023-06-28 Thread via GitHub
gnodet commented on code in PR #1185: URL: https://github.com/apache/maven/pull/1185#discussion_r1245404521 ## maven-model-builder/src/main/java/org/apache/maven/model/io/DefaultModelReader.java: ## @@ -120,23 +120,29 @@ private Model read(Reader reader, Path pomFile, Map

[GitHub] [maven] gnodet commented on a diff in pull request #1185: [MNG-7830] Switch from plexus-xml to stax / woodstox

2023-06-28 Thread via GitHub
gnodet commented on code in PR #1185: URL: https://github.com/apache/maven/pull/1185#discussion_r1245206728 ## maven-model-transform/pom.xml: ## @@ -32,6 +32,14 @@ under the License. org.codehaus.plexus plexus-xml + + org.codehaus.woodstox +

[GitHub] [maven] gnodet commented on a diff in pull request #1185: [MNG-7830] Switch from plexus-xml to stax / woodstox

2023-06-28 Thread via GitHub
gnodet commented on code in PR #1185: URL: https://github.com/apache/maven/pull/1185#discussion_r1245205028 ## maven-model/src/test/java/org/apache/maven/model/v4/StaxTest.java: ## @@ -0,0 +1,104 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more

[GitHub] [maven] gnodet commented on a diff in pull request #1185: [MNG-7830] Switch from plexus-xml to stax / woodstox

2023-06-28 Thread via GitHub
gnodet commented on code in PR #1185: URL: https://github.com/apache/maven/pull/1185#discussion_r1245204583 ## maven-model/src/test/java/org/apache/maven/model/v4/StaxTest.java: ## @@ -0,0 +1,104 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more