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

Prevent empty folder from being created when dumping JSON #382

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

jonas-resch
Copy link
Contributor

At the moment an empty folder will be created, even when executing the command with the --dump-json flag.

@jonas-resch
Copy link
Contributor Author

Hi @SathyaBhat,

I looked into why the pipelines are failing but I can't seem to get why. Yt-dlp fails when downloading the test file (ERROR: [youtube] orTosjJUQ3k: Sign in to confirm you’re not a bot. This helps protect our community. Learn more).

Yt-dlp seems to be up to date: Requirement already satisfied: yt-dlp>=2023.3.4 in /opt/hostedtoolcache/Python/3.11.10/x64/lib/python3.11/site-packages (from spotify_dl==8.9.0) (2024.10.22).

Did you encounter this error before?

Thanks.

@SathyaBhat
Copy link
Owner

Hi @SathyaBhat,

I looked into why the pipelines are failing but I can't seem to get why. Yt-dlp fails when downloading the test file (ERROR: [youtube] orTosjJUQ3k: Sign in to confirm you’re not a bot. This helps protect our community. Learn more).

Yt-dlp seems to be up to date: Requirement already satisfied: yt-dlp>=2023.3.4 in /opt/hostedtoolcache/Python/3.11.10/x64/lib/python3.11/site-packages (from spotify_dl==8.9.0) (2024.10.22).

Did you encounter this error before?

Thanks.

youtube can block access to yt-dlp, I haven't found an easy workaround yet

@jonas-resch
Copy link
Contributor Author

Thanks, I'll retry the pipeline every now and then so that it at least runs through one time with the new changes.

Jonas Resch added 4 commits November 10, 2024 14:20
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