Thanks to visit codestin.com
Credit goes to github.com

Skip to content

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

Merged
chrisknoll merged 2 commits into
OHDSI:ohdsi-removed-dependency-on-commons-langfrom
ecruz165:refactor/removed-dependency-on-commons-lang
Nov 27, 2024
Merged

chrisknoll merged 2 commits into
OHDSI:ohdsi-removed-dependency-on-commons-langfrom
ecruz165:refactor/removed-dependency-on-commons-lang

Conversation

@ecruz165

Copy link
Copy Markdown
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
Copy Markdown
Collaborator

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

@anthonysena

Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
Collaborator

Associating with OHDSI/StandardizedAnalysisAPI#54

@anthonysena

Copy link
Copy Markdown
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!

@chrisknoll
chrisknoll changed the base branch from master to ohdsi-removed-dependency-on-commons-lang November 27, 2024 14:24
@chrisknoll
chrisknoll merged commit 19b615b into OHDSI:ohdsi-removed-dependency-on-commons-lang Nov 27, 2024
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.

3 participants