[GitHub] maven-surefire issue #168: [SUREFIRE-1424] javax.transaction.TransactionMana...

2017-12-02 Thread Tibor17
Github user Tibor17 commented on the issue: https://github.com/apache/maven-surefire/pull/168 @eolivelli Thx for contributing. Close this PR. It was pushed to master. --- - To unsubscribe, e-mail:

[GitHub] maven-surefire issue #168: [SUREFIRE-1424] javax.transaction.TransactionMana...

2017-12-02 Thread eolivelli
Github user eolivelli commented on the issue: https://github.com/apache/maven-surefire/pull/168 @Tibor17 Ihave squashed the commits and reworded the commit msg. --- - To unsubscribe, e-mail:

[GitHub] maven-surefire issue #168: [SUREFIRE-1424] javax.transaction.TransactionMana...

2017-12-02 Thread eolivelli
Github user eolivelli commented on the issue: https://github.com/apache/maven-surefire/pull/168 Ok, will do this afternoon. Thanks --- - To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org For additional

[GitHub] maven-surefire issue #168: [SUREFIRE-1424] javax.transaction.TransactionMana...

2017-12-02 Thread Tibor17
Github user Tibor17 commented on the issue: https://github.com/apache/maven-surefire/pull/168 The comment `[SUREFIRE-1424] javax.transaction.TransactionManager not visible with Java9` should be in commit. --- - To

[GitHub] maven-surefire issue #168: [SUREFIRE-1424] javax.transaction.TransactionMana...

2017-12-02 Thread Tibor17
Github user Tibor17 commented on the issue: https://github.com/apache/maven-surefire/pull/168 If you could squash these two commits in one, it would be great. --- - To unsubscribe, e-mail:

[GitHub] maven-surefire issue #168: [SUREFIRE-1424] javax.transaction.TransactionMana...

2017-12-01 Thread eolivelli
Github user eolivelli commented on the issue: https://github.com/apache/maven-surefire/pull/168 @Tibor17 please check it out --- - To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org For additional commands,

[GitHub] maven-surefire issue #168: [SUREFIRE-1424] javax.transaction.TransactionMana...

2017-12-01 Thread eolivelli
Github user eolivelli commented on the issue: https://github.com/apache/maven-surefire/pull/168 Oh, got it, I have to run "mvn site" on the root of the project not only on maven-surefire-plugin --- - To

[GitHub] maven-surefire issue #168: [SUREFIRE-1424] javax.transaction.TransactionMana...

2017-12-01 Thread eolivelli
Github user eolivelli commented on the issue: https://github.com/apache/maven-surefire/pull/168 @Tibor17 I am trying to update the site, how can I rebuild the site locally ? I am trying with "mvn site" but it fails ---

[GitHub] maven-surefire issue #168: [SUREFIRE-1424] javax.transaction.TransactionMana...

2017-11-29 Thread eolivelli
Github user eolivelli commented on the issue: https://github.com/apache/maven-surefire/pull/168 @Tibor17 ok I will do this change. Will send tomorrow. --- - To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org

[GitHub] maven-surefire issue #168: [SUREFIRE-1424] javax.transaction.TransactionMana...

2017-11-29 Thread Tibor17
Github user Tibor17 commented on the issue: https://github.com/apache/maven-surefire/pull/168 @eolivelli We need to add a new chapter on the top in https://github.com/apache/maven-surefire/blob/master/maven-surefire-plugin/src/site/markdown/java9.md and add the following API.

[GitHub] maven-surefire issue #168: [SUREFIRE-1424] javax.transaction.TransactionMana...

2017-11-28 Thread Tibor17
Github user Tibor17 commented on the issue: https://github.com/apache/maven-surefire/pull/168 @eolivelli I will have a look. I was busy last week. --- - To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org

[GitHub] maven-surefire issue #168: [SUREFIRE-1424] javax.transaction.TransactionMana...

2017-11-23 Thread eolivelli
Github user eolivelli commented on the issue: https://github.com/apache/maven-surefire/pull/168 @Tibor17 I have rewritten the patch and address your comment of not altering the docs. Tell me the next step in your vision, I will be happy to move forward and finish this important

[GitHub] maven-surefire issue #168: [SUREFIRE-1424] javax.transaction.TransactionMana...

2017-11-22 Thread eolivelli
Github user eolivelli commented on the issue: https://github.com/apache/maven-surefire/pull/168 Ok will do asap --- - To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org For additional commands, e-mail:

[GitHub] maven-surefire issue #168: [SUREFIRE-1424] javax.transaction.TransactionMana...

2017-11-22 Thread Tibor17
Github user Tibor17 commented on the issue: https://github.com/apache/maven-surefire/pull/168 @eolivelli I have finished my changes in master. Can you continue? --- - To unsubscribe, e-mail:

[GitHub] maven-surefire issue #168: [SUREFIRE-1424] javax.transaction.TransactionMana...

2017-11-19 Thread eolivelli
Github user eolivelli commented on the issue: https://github.com/apache/maven-surefire/pull/168 No problem, the change was minimal, I will start from scratch --- - To unsubscribe, e-mail:

[GitHub] maven-surefire issue #168: [SUREFIRE-1424] javax.transaction.TransactionMana...

2017-11-19 Thread Tibor17
Github user Tibor17 commented on the issue: https://github.com/apache/maven-surefire/pull/168 It was done. But I don't think you can easily rebase it since some classes were deleted or methods moved. --- - To

[GitHub] maven-surefire issue #168: [SUREFIRE-1424] javax.transaction.TransactionMana...

2017-11-19 Thread eolivelli
Github user eolivelli commented on the issue: https://github.com/apache/maven-surefire/pull/168 Sure, I will rebase once you merged your big changes --- - To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org For