Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

QuickFix: Upgrade commons-lang to commons-text to fix deprecation #750

Merged

Conversation

vitorpamplona
Copy link
Contributor

@vitorpamplona vitorpamplona commented May 20, 2022

Classes below are deprecated and will soon be removed from apache.commons.lang3.

  • CharSequenceTranslator
  • AggregateTranslator
  • UnicodeUnescaper
  • LookupTranslator
  • JavaUnicodeEscaper

They were moved to apache.commons.text

@JPercival
Copy link
Contributor

Resolves #698

@JPercival JPercival requested a review from brynrhodes June 3, 2022 17:42
@JPercival
Copy link
Contributor

@brynrhodes - This is a quick one. Can we get a review on it?

@JPercival
Copy link
Contributor

Bryn's at DevDays. @mdnazmulkarim? Can I get a second review on this?

@JPercival
Copy link
Contributor

@vitorpamplona - I see some conflicts. Can you resolve and I'll merge?

…ns-text-upgrade

# Conflicts:
#	Src/java/model/build.gradle
@vitorpamplona
Copy link
Contributor Author

@JPercival ready to go here too.

@JPercival JPercival merged commit 325e00f into cqframework:master Jun 8, 2022
@vitorpamplona vitorpamplona deleted the commons-lang-to-commons-text-upgrade branch June 8, 2022 18:35
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.

None yet

3 participants