Skip to content

[MNG-6829] Replace any StringUtils#isEmpty(String) and #isNotEmpty(String)#214

Merged
elharo merged 1 commit intoapache:masterfrom
timtebeek:refactor/replace-any-string-utils-is-empty-string-and-is-not-empty-string
May 12, 2023
Merged

[MNG-6829] Replace any StringUtils#isEmpty(String) and #isNotEmpty(String)#214
elharo merged 1 commit intoapache:masterfrom
timtebeek:refactor/replace-any-string-utils-is-empty-string-and-is-not-empty-string

Conversation

@timtebeek
Copy link
Copy Markdown
Contributor

@michael-o
Copy link
Copy Markdown
Member

Buit not gone from the POM, no?

@timtebeek
Copy link
Copy Markdown
Contributor Author

Buit not gone from the POM, no?

We're not at the stage yet where we have removed all methods of StringUtils, so we can't yet remove the dependency. See for instance

p.setPath(StringUtils.join(
mavenProject.getCompileClasspathElements().iterator(), File.pathSeparator));

As requested I'm doing small individual (automated) changes, such that they are easier to review and merge, with less risk.
We hope to get to fully removing StringUtils soon, but feel free to help out either in an automated fashion, or by cleaning up what remains after the first few rounds.

@elharo elharo merged commit 1dcccaf into apache:master May 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants