maven pull request: [MNG-5075] MavenProject.getParent throws undocumented I...

2013-04-29 Thread jglick
GitHub user jglick opened a pull request: https://github.com/apache/maven/pull/8 [MNG-5075] MavenProject.getParent throws undocumented ISE Fix for [MNG-5075](http://jira.codehaus.org/browse/MNG-5075): `MavenProject.getParent` throws `IllegalStateException` when there is a model

maven pull request: [MNG-5075] MavenProject.getParent throws undocumented I...

2014-01-06 Thread jglick
Github user jglick closed the pull request at: https://github.com/apache/maven/pull/8 - To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org For additional commands, e-mail: dev-h...@maven.apache.org

[GitHub] maven-enforcer pull request #23: Adding RequireUpperBoundDeps.excludes optio...

2017-06-12 Thread jglick
GitHub user jglick opened a pull request: https://github.com/apache/maven-enforcer/pull/23 Adding RequireUpperBoundDeps.excludes option Needed to implement [JENKINS-41631](https://issues.jenkins-ci.org/browse/JENKINS-41631): there are certain known cases (that for various reasons

[GitHub] maven-enforcer pull request #22: Site fixes

2017-06-12 Thread jglick
GitHub user jglick opened a pull request: https://github.com/apache/maven-enforcer/pull/22 Site fixes Some fixes I found desirable or necessary when testing site generation locally. @reviewbybees for my colleagues You can merge this pull request into a Git repository

[GitHub] maven-enforcer pull request #22: [MENFORCER-272] Site fixes

2017-06-15 Thread jglick
Github user jglick closed the pull request at: https://github.com/apache/maven-enforcer/pull/22 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature

[GitHub] maven-enforcer pull request #23: [MENFORCER-273] Adding RequireUpperBoundDep...

2017-06-15 Thread jglick
Github user jglick closed the pull request at: https://github.com/apache/maven-enforcer/pull/23 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature

[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

[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 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_r153657657 --- Diff: core-it-suite/src/test/java/org/apache/maven/it/MavenIT0199CyclicImportScopeTest.java --- @@ -0,0 +1,56 @@ +package