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

build(deps): Sanitize dependencies based on dependency:analyze-report results #7294

Merged
merged 2 commits into from
Jan 11, 2025

Conversation

aikebah
Copy link
Collaborator

@aikebah aikebah commented Jan 8, 2025

Description of Change

Processed the various dependency-analysis reports for missing / superfluous dependencies, so that now the poms also reflect the various used-undeclared usages of transitive dependencies.

Includes a code-change to upgrade the last remnants of commons-collections to commons-collections4.

The only items I left 'unfixed' are the unused cases for the various compile/codescan annotations libraries and junit/hamcrest.

Related issues

N/A

Have test cases been added to cover the new functionality?

N/A

@boring-cyborg boring-cyborg bot added ant changes to ant cli changes to the cli core changes to core maven changes to the maven plugin utils changes to utils labels Jan 8, 2025
jeremylong
jeremylong previously approved these changes Jan 11, 2025
Copy link
Owner

@jeremylong jeremylong left a comment

Choose a reason for hiding this comment

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

LGTM

@jeremylong jeremylong added this to the 12.0.0 milestone Jan 11, 2025
@jeremylong jeremylong merged commit 5a47091 into main Jan 11, 2025
8 checks passed
@jeremylong jeremylong deleted the chore/dependencies branch January 11, 2025 13:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ant changes to ant cli changes to the cli core changes to core maven changes to the maven plugin utils changes to utils
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants