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

Doesn't search song on Spotify if "--no-metadata" passed #404

Merged
merged 3 commits into from
Oct 23, 2018
Merged

Doesn't search song on Spotify if "--no-metadata" passed #404

merged 3 commits into from
Oct 23, 2018

Conversation

manveerxyz
Copy link
Contributor

Solves #263

I just added simple checks to const.args.no_metadata in places where metadata is gathered. As usual, let me know if there's a better way!

@manveerxyz
Copy link
Contributor Author

For some reason the Travis-CI build isn't triggering, or it just isn't showing up on my end. Can someone with write access trigger a rebuild? Or would I have to close/re-open the PR?

@linusg
Copy link
Contributor

linusg commented Oct 22, 2018

I can trigger builds for branches (e. g. Master), but that won't get us anywhere. Seems like Travis missed this PR at all: https://travis-ci.org/ritiek/spotify-downloader/pull_requests

GH had some major issues in the last 24 hours, that might be the reason (check their Status page). Not sure what we should do here, TBH... Of course you can run the tests locally.

@manveerxyz
Copy link
Contributor Author

Oh I see, yeah I when I tried to view this PR about 2 hours ago I only got a 404 page. I'll try running the tests locally when I have time.

@ritiek
Copy link
Member

ritiek commented Oct 22, 2018

when I tried to view this PR about 2 hours ago I only got a 404 page.

What else would you expect from #404? ;)

I'll take a better look at this PR tomorrow when hopefully GitHub gets stable.

Copy link
Member

@ritiek ritiek left a comment

Choose a reason for hiding this comment

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

Looks good. I'll merge it. Thanks @ManveerBasra!

@ritiek ritiek merged commit 94f0b3e into spotDL:master Oct 23, 2018
@ritiek ritiek added this to the v1.1.0 milestone Nov 13, 2018
@ritiek ritiek mentioned this pull request Nov 13, 2018
6 tasks
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.

None yet

3 participants