[GitHub] [maven-surefire] kwin commented on a diff in pull request #667: [SUREFIRE-2179] Support adding additional Maven dependencies to the test classpath

2023-06-28 Thread via GitHub
kwin commented on code in PR #667: URL: https://github.com/apache/maven-surefire/pull/667#discussion_r1243772435 ## maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/AbstractSurefireMojo.java: ## @@ -281,6 +285,21 @@ public abstract class AbstractSurefireMojo

[GitHub] [maven-surefire] kwin commented on a diff in pull request #667: [SUREFIRE-2179] Support adding additional Maven dependencies to the test classpath

2023-06-27 Thread via GitHub
kwin commented on code in PR #667: URL: https://github.com/apache/maven-surefire/pull/667#discussion_r1243773343 ## maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/AbstractSurefireMojo.java: ## @@ -3474,6 +3552,14 @@ public void

[GitHub] [maven-surefire] kwin commented on a diff in pull request #667: [SUREFIRE-2179] Support adding additional Maven dependencies to the test classpath

2023-06-27 Thread via GitHub
kwin commented on code in PR #667: URL: https://github.com/apache/maven-surefire/pull/667#discussion_r1243772435 ## maven-surefire-common/src/main/java/org/apache/maven/plugin/surefire/AbstractSurefireMojo.java: ## @@ -281,6 +285,21 @@ public abstract class AbstractSurefireMojo