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

NUTCH-3092 Replace all imports of commons-lang by commons-lang3 #839

Merged

Conversation

sebastian-nagel
Copy link
Contributor

No description provided.

Copy link
Member

@lewismc lewismc left a comment

Choose a reason for hiding this comment

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

+1
Good catch . I wouldn't have seen this.

@sebastian-nagel
Copy link
Contributor Author

Rebased to master.

@sebastian-nagel sebastian-nagel force-pushed the NUTCH-3092-commons-lang branch from d87ec93 to 6075003 Compare December 3, 2024 11:45
@sebastian-nagel
Copy link
Contributor Author

Ok, but the rebase uncovered an issue introduced with NUTCH-3094 / #841 - although there are plenty of changes in the Java code (core and plugins), no unit tests are run. I'll try to address this before merging this PR.

@sebastian-nagel sebastian-nagel force-pushed the NUTCH-3092-commons-lang branch from 6075003 to a0b0c9b Compare December 3, 2024 12:56
@sebastian-nagel sebastian-nagel merged commit e2a29d0 into apache:master Dec 3, 2024
4 checks passed
@sebastian-nagel sebastian-nagel deleted the NUTCH-3092-commons-lang branch December 4, 2024 17:54
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.

2 participants