[GitHub] maven-integration-testing pull request #25: Importing a BOM works even when ...

2017-11-28 Thread jglick
Github user jglick commented on a diff in the pull request: https://github.com/apache/maven-integration-testing/pull/25#discussion_r153657657 --- Diff: core-it-suite/src/test/java/org/apache/maven/it/MavenIT0199CyclicImportScopeTest.java --- @@ -0,0 +1,56 @@ +package

[GitHub] maven-integration-testing pull request #25: Importing a BOM works even when ...

2017-11-28 Thread jglick
Github user jglick commented on a diff in the pull request: https://github.com/apache/maven-integration-testing/pull/25#discussion_r153657727 --- Diff: core-it-suite/src/test/java/org/apache/maven/it/MavenIT0199CyclicImportScopeTest.java --- @@ -0,0 +1,56 @@ +package

[GitHub] maven-integration-testing pull request #25: Importing a BOM works even when ...

2017-11-28 Thread jglick
Github user jglick commented on a diff in the pull request: https://github.com/apache/maven-integration-testing/pull/25#discussion_r153657505 --- Diff: core-it-suite/src/test/java/org/apache/maven/it/IntegrationTestSuite.java --- @@ -654,6 +654,7 @@ public static Test suite()

[GitHub] maven-integration-testing pull request #25: Importing a BOM works even when ...

2017-11-28 Thread jglick
GitHub user jglick opened a pull request: https://github.com/apache/maven-integration-testing/pull/25 Importing a BOM works even when it has the same components in dep mgmt Does not correspond to any known bug, but it seems like a potentially unusual practice deserving of test