[GitHub] [maven-pmd-plugin] timtebeek commented on pull request #124: [MNG-6829] Replace any StringUtils#isEmpty(String) and #isNotEmpty(String)

["timtebeek (via GitHub)" <gi...@apache.org>]

timtebeek commented on PR #124:
URL: https://github.com/apache/maven-pmd-plugin/pull/124#issuecomment-1539062036

   > In fact, now that I think about it, we should pause these changes until Moderne does sign the corporate CLA. An Individual CLA is insufficient given the co-authorship line.
   
   @elharo I'm pleased to say we've finally been able to sign (and get accepted) the CCLA:
   
   > This message acknowledges receipt of the following document, which has been filed in the Apache Software Foundation records:
   >  CCLA from Moderne Inc.
   
   Would you want me to resume creating pull requests to replace `StringUtils#is/Not/Empty(String)` as we've done here?
   I was thinking to open up to ten at a time, and only start a next batch when there's less than five open; does that sound OK?
   
   The changes would be made by running this recipe against the Apache Maven organization defined in the top right corner:
   https://public.moderne.io/recipes/org.openrewrite.java.migrate.apache.commons.lang.IsNotEmptyToJdk
   You're more than welcome to have a look as well; shouldn't take more than a couple minutes to create a few PRs or direct commits.


-- 
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-unsubscribe@maven.apache.org

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