[GitHub] [maven-surefire] Tibor17 commented on pull request #470: [SUREFIRE-2012] Use maven-shared-utils instead of surefire-shared-utils in Report Parser. Removed commons-lang in Report Plugin.

2022-02-14 Thread GitBox


Tibor17 commented on pull request #470:
URL: https://github.com/apache/maven-surefire/pull/470#issuecomment-1039169802


   @slawekjaranowski 
   Pls check it now. The codestyle XMl did not make it. The problem was with 
`.editorconfig`.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@maven.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [maven-surefire] Tibor17 commented on pull request #470: [SUREFIRE-2012] Use maven-shared-utils instead of surefire-shared-utils in Report Parser. Removed commons-lang in Report Plugin.

2022-02-14 Thread GitBox


Tibor17 commented on pull request #470:
URL: https://github.com/apache/maven-surefire/pull/470#issuecomment-1039100651


   @slawekjaranowski 
   Anyway, the class `org.apache.maven.shared.utils.io.DirectoryScanner` in 
m-shared-u is deprecated. It can be substituted by 
`java.nio.file.DirectoryStream`. We should use our own scanner class.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@maven.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [maven-surefire] Tibor17 commented on pull request #470: [SUREFIRE-2012] Use maven-shared-utils instead of surefire-shared-utils in Report Parser. Removed commons-lang in Report Plugin.

2022-02-14 Thread GitBox


Tibor17 commented on pull request #470:
URL: https://github.com/apache/maven-surefire/pull/470#issuecomment-1039068707


   @slawekjaranowski 
   `surefire-shared-utils` is not necessary in the Report plugin. Using 
`maven-shared-utils` instead.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@maven.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org