[GitHub] maven-surefire issue #143: [SUREFIRE-1416] maven-surefire-parser: add new me...

2018-01-03 Thread Tibor17
Github user Tibor17 commented on the issue:

https://github.com/apache/maven-surefire/pull/143
  
@surli 
Thx for contributing. Now you can close the PR.


---


[GitHub] maven-surefire issue #143: [SUREFIRE-1416] maven-surefire-parser: add new me...

2018-01-03 Thread surli
Github user surli commented on the issue:

https://github.com/apache/maven-surefire/pull/143
  
@Tibor17 I can confirm it's working for me and it's useful! See my PR 
https://github.com/Spirals-Team/librepair/pull/380


---


[GitHub] maven-surefire issue #143: [SUREFIRE-1416] maven-surefire-parser: add new me...

2018-01-03 Thread Tibor17
Github user Tibor17 commented on the issue:

https://github.com/apache/maven-surefire/pull/143
  
@surli 
Pls try to use it in your projects. I want to know if it is useful for you 
and working. Thx.


---


[GitHub] maven-surefire issue #143: [SUREFIRE-1416] maven-surefire-parser: add new me...

2018-01-03 Thread surli
Github user surli commented on the issue:

https://github.com/apache/maven-surefire/pull/143
  
Hi @Tibor17 

sorry I was off lately so I just saw your comments. I did not check what 
you asked before. I'm currently reviewing your code and it looks fine for me :) 
Thanks!


---


[GitHub] maven-surefire issue #143: [SUREFIRE-1416] maven-surefire-parser: add new me...

2018-01-03 Thread Tibor17
Github user Tibor17 commented on the issue:

https://github.com/apache/maven-surefire/pull/143
  
@surli 
If there are no objections, I would push the branch SUREFIRE-1416 to master.
Thx.


---


[GitHub] maven-surefire issue #143: [SUREFIRE-1416] maven-surefire-parser: add new me...

2017-12-27 Thread Tibor17
Github user Tibor17 commented on the issue:

https://github.com/apache/maven-surefire/pull/143
  
@surli 
I had to change you code in new branch

https://git1-us-west.apache.org/repos/asf?p=maven-surefire.git;a=shortlog;h=refs/heads/SUREFIRE-1416
Can you build the version `2.21.0-SNAPSHOT` using `mvn install -DskipTests` 
and let me know if the report site works for you? I want to include this fix in 
release in several days.


---