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

fix(deps): Deprecate old commmons-lang library #2750

Merged
merged 1 commit into from
Dec 19, 2024

Conversation

heliocastro
Copy link
Contributor

Old apache-commons-lang was puled down by nouveau-handler, and misleading outdated code in different sources.
All code now is adjusted to commons-lang3 or commons-text where needed.

@heliocastro heliocastro self-assigned this Nov 26, 2024
@heliocastro heliocastro added ready ready to merge WIP work in progress and removed ready ready to merge labels Nov 26, 2024
@GMishx GMishx added this to the Release - 20.0.0 milestone Nov 27, 2024
@heliocastro heliocastro force-pushed the fix/deprecate_old_commons branch from 0710bad to cd9201c Compare December 19, 2024 22:26
@heliocastro heliocastro removed the WIP work in progress label Dec 19, 2024
Signed-off-by: Helio Chissini de Castro <heliocastro@gmail.com>
@heliocastro heliocastro force-pushed the fix/deprecate_old_commons branch from cd9201c to 9f3c99c Compare December 19, 2024 22:38
@heliocastro heliocastro added the ready ready to merge label Dec 19, 2024
@heliocastro heliocastro merged commit 325cf0e into main Dec 19, 2024
3 checks passed
@heliocastro heliocastro deleted the fix/deprecate_old_commons branch December 19, 2024 22:50
@GMishx
Copy link
Member

GMishx commented Dec 20, 2024

@heliocastro This PR was changed from WIP (thus not reviewed) to ready and self-merged without any review from anyone other than yourself. This does not follow the guidelines set for the SW360 project: https://eclipse.dev/sw360/docs/development/dev-dod-and-style/

We have discussed the same multiple times as a bad practice.

@heliocastro
Copy link
Contributor Author

heliocastro commented Dec 20, 2024 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants