[ 
https://issues.apache.org/jira/browse/SUREFIRE-2172?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17872365#comment-17872365
 ] 

ASF GitHub Bot commented on SUREFIRE-2172:
------------------------------------------

FredrikAnderson commented on PR #768:
URL: https://github.com/apache/maven-surefire/pull/768#issuecomment-2277953892

   Removed usage of StringUtils from: 
   * Apache Commons Lang StringUtils
   * Surefire API StringUtils
   
   Now only using Surefire Shared StringUtils. Is this enough for this ticket 
or are we aiming for something else? 
   




> StringUtils: yet more of them
> -----------------------------
>
>                 Key: SUREFIRE-2172
>                 URL: https://issues.apache.org/jira/browse/SUREFIRE-2172
>             Project: Maven Surefire
>          Issue Type: Improvement
>            Reporter: Elliotte Rusty Harold
>            Priority: Minor
>              Labels: up-for-grabs
>
> This project contains not one, not two, but **three** different StringUtils 
> classes that duplicate functionality in the JDK and elsewhere. One or two of 
> them imght be built by shading other classes. They're even sometimes both 
> imported in the same classes such as LegacyPojoStackTraceWriter:
> import org.apache.maven.surefire.api.util.internal.StringUtils;
> import static org.apache.maven.surefire.shared.utils.StringUtils.isNotEmpty;
> Figure out what to do about this. 



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to