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

Refactored imports to depend on org.apache.commons:commons-lang3 #2215

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

ecruz165
Copy link
Contributor

I saw we are using in code references to commons-lang and commons-lang3. I removed dependency on commons-lang, removed any unused imports in files i updated. And I introduced commons-text to support usage of WordUtils that was moved to that library.

@ecruz165 ecruz165 changed the title Refactor/removed dependency on commons lang Refactored imports to depend on commons-lang3 Feb 25, 2023
@ecruz165 ecruz165 changed the title Refactored imports to depend on commons-lang3 Refactored imports to depend on org.apache.commons:commons-lang3 Feb 25, 2023
@anthonysena anthonysena added this to the v2.14 milestone Apr 4, 2023
@anthonysena
Copy link
Collaborator

These changes are related to upgrading to Spring Boot 2.x

@anthonysena
Copy link
Collaborator

@ecruz165 - can you resolve the conflicts and then we can merge this into master?

@anthonysena anthonysena modified the milestones: v2.14, v2.15 Sep 19, 2023
@anthonysena
Copy link
Collaborator

Discussion on this from the Atlas WG - we'll revisit this in the next release when the SOLR work is completed since there is a merge conflict on that file at the moment.

@anthonysena
Copy link
Collaborator

Associating with OHDSI/StandardizedAnalysisAPI#54

@anthonysena
Copy link
Collaborator

@ecruz165 - just following up here to see if you can resolve the merge conflicts? We can the put this into the v2.15 release. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Status: Pushed To Next Release
Development

Successfully merging this pull request may close these issues.

2 participants