Skip to content

Fix empty dependency exclusion filtering#1651

Closed
Aayush10016 wants to merge 1 commit into
apache:masterfrom
Aayush10016:fix/issue-1645-empty-excludes
Closed

Fix empty dependency exclusion filtering#1651
Aayush10016 wants to merge 1 commit into
apache:masterfrom
Aayush10016:fix/issue-1645-empty-excludes

Conversation

@Aayush10016

Copy link
Copy Markdown

Summary

  • return an empty result when dependency exclusion patterns are absent
  • stop verbose analysis output from reporting real findings again as ignored dependencies
  • add integration assertions for both used-undeclared and unused-declared output

Fixes #1645.

Root cause

filterDependencies(...) returned its full input set when the exclusion array was null or empty. Callers treat the return value as the dependencies removed by an exclusion filter, so the full input was incorrectly added to the ignored-dependency sections.

Verification

  • regression assertions failed before the production change and passed afterward
  • mvn -Prun-its -Dinvoker.test=analyze verify — passed
  • mvn -Prun-its verify — passed; 99 integration projects, 0 failures or errors
  • unit tests, Checkstyle, Spotless, Apache RAT, compilation, and packaging passed as part of the full build
  • git diff --check — passed

@elharo elharo closed this Jul 7, 2026
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.

filterDependencies returns the full input set when no excludes are configured, polluting verbose output

2 participants