Skip to content

Upgrade Commons Lang from 2 to 3#284

Merged
strangelookingnerd merged 1 commit into
masterfrom
remove_lang2
Jul 31, 2025
Merged

Upgrade Commons Lang from 2 to 3#284
strangelookingnerd merged 1 commit into
masterfrom
remove_lang2

Conversation

@strangelookingnerd
Copy link
Copy Markdown
Contributor

Migrate from deprecated/EOL Commons Lang 2 to Commons Lang 3.

Testing done

mvn clean verify

Submitter checklist

  • Make sure you are opening from a topic/feature/bugfix branch (right side) and not your main branch!
  • Ensure that the pull request title represents the desired changelog entry
  • Please describe what you did
  • Link to relevant issues in GitHub or Jira
  • Link to relevant pull requests, esp. upstream and downstream changes
  • Ensure you have provided tests that demonstrate the feature works or the issue is fixed

@strangelookingnerd strangelookingnerd requested a review from a team as a code owner July 31, 2025 13:48
@strangelookingnerd strangelookingnerd added dependencies Pull requests that update a dependency file chore Reduce maintenance labels Jul 31, 2025
@strangelookingnerd strangelookingnerd enabled auto-merge (rebase) July 31, 2025 13:57
@strangelookingnerd strangelookingnerd merged commit bf2caf5 into master Jul 31, 2025
19 checks passed
@strangelookingnerd strangelookingnerd deleted the remove_lang2 branch July 31, 2025 14:01
import java.io.Serial;
import org.apache.commons.lang.StringEscapeUtils;
import org.apache.commons.lang.StringUtils;
import org.apache.commons.lang3.StringEscapeUtils;
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Note that this is deprecated:

Deprecated.
As of 3.6, use Apache Commons Text StringEscapeUtils instead

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah, deprecation notice did not pop up in my IDE. Will update in a separate PR. Thanks for letting me know!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

chore Reduce maintenance dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants