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

Fixed '[WARNING] There were 0 download errors - check the warnings above to not log warning unless download errors > 0 #402

Merged
merged 4 commits into from
Jun 23, 2023

Conversation

blekinge
Copy link
Contributor

Really not much to say here. It is just annoying to see the warning in the maven log when where really is nothing to warn about

default:
throw new IllegalStateException( "Unexpected value of " + ErrorRemedy.class.getName() + ": "
+ errorRemedy );
if ( downloadErrorCount > 0 ) {
Copy link
Contributor

Choose a reason for hiding this comment

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

double check formatting

@george-kay
Copy link

george-kay commented Mar 14, 2022

I can't find the change in latest sources. Could someone approve it? @nfalco79

@nfalco79
Copy link
Contributor

I'm not the owner of this plugin so my approval count 0

@george-kay
Copy link

@ppalaga could you approve it? It bugs our automation with this error message. Or just tell us what todo. Thx a bunch!

@FlorianWolters
Copy link

Can this be merged please? The warning is still occuring with v2.1.0:

[INFO] --- license:2.1.0:download-licenses (download-licenses) @ hello_world ---
[WARNING] There were 0 download errors - check the warnings above

@slawekjaranowski slawekjaranowski self-assigned this Jun 20, 2023
@slawekjaranowski slawekjaranowski merged commit e244187 into mojohaus:master Jun 23, 2023
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.

5 participants