Skip to content

fix: Also apply muteNoisyLoggers to UpdateMojo#7469

Merged
jeremylong merged 1 commit intodependency-check:mainfrom
qsiebers:main
Feb 25, 2025
Merged

fix: Also apply muteNoisyLoggers to UpdateMojo#7469
jeremylong merged 1 commit intodependency-check:mainfrom
qsiebers:main

Conversation

@qsiebers
Copy link
Copy Markdown
Contributor

Description of Change

The update-only mojo also has some log spamming that can be disabled by using BaseDependencyCheckMojo.muteNoisyLoggers

Related issues

None as far as I know or found

Have test cases been added to cover the new functionality?

Manually tested, not sure if there is a test for log spam

@boring-cyborg boring-cyborg bot added the maven changes to the maven plugin label Feb 25, 2025
@qsiebers qsiebers changed the title Also apply muteNoisyLoggers to UpdateMojo feat: Also apply muteNoisyLoggers to UpdateMojo Feb 25, 2025
@qsiebers qsiebers changed the title feat: Also apply muteNoisyLoggers to UpdateMojo fix: Also apply muteNoisyLoggers to UpdateMojo Feb 25, 2025
Copy link
Copy Markdown
Collaborator

@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.1.1 milestone Feb 25, 2025
@jeremylong jeremylong merged commit 99694c1 into dependency-check:main Feb 25, 2025
2 of 3 checks passed
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 28, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

maven changes to the maven plugin

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants