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

Added a missing switch to the fetchart command #2683

Merged
merged 2 commits into from
Sep 5, 2017
Merged

Conversation

euri10
Copy link
Contributor

@euri10 euri10 commented Sep 5, 2017

I was trying to complete my album art collection today and found it not very friendly: maybe I'm using it the wrong way, there's always that possibility!

In the current state, issuing a beet fetchart command yields a gigantic list of already found stuff I really don't care about when your goal is just to fill the gaps in your collection,

that PR may solve the "issue"

@sampsyo
Copy link
Member

sampsyo commented Sep 5, 2017

This looks great! Yes, that can be annoying.

What do you think about calling this something like "quiet" (-q) to convey that it really only affects the logging, not the actual art-fetching functionality?

Also, would you mind writing a quick changelog entry?

@sampsyo
Copy link
Member

sampsyo commented Sep 5, 2017

Perfect. Thanks again! ✨ 🚀

@sampsyo sampsyo merged commit cb2cb00 into beetbox:master Sep 5, 2017
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.

2 participants