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

Dry run for downloads #37

Closed
Owez opened this issue Jan 2, 2023 · 2 comments · Fixed by #38
Closed

Dry run for downloads #37

Owez opened this issue Jan 2, 2023 · 2 comments · Fixed by #38
Assignees
Labels
enhancement New feature or request

Comments

@Owez
Copy link
Owner

Owez commented Jan 2, 2023

If there's a massive channel and you just want to download the videos after stopping the big download session, you probably don't want to refresh your massive metadata. To fix this, having some --dry argument in the refresh config would be good. Need to differentiate:

  • Not pulling new metadata (this issue)
  • Not downloading videos (future issue)

So that these two options wont collide with naming. I don't want --no-.. because that's a bit ugly.

@Owez Owez added the enhancement New feature or request label Jan 2, 2023
@Owez Owez self-assigned this Jan 2, 2023
@Owez
Copy link
Owner Author

Owez commented Jan 2, 2023

This will be part of the 1.1.1 release, I've thought about it a bit and --no.. is the best of the bad options. The problem is the future issue with "dont download videos" could be done right now with just:

yark refresh x --videos=0

@Owez Owez linked a pull request Jan 2, 2023 that will close this issue
@Owez Owez closed this as completed in #38 Jan 2, 2023
@Owez
Copy link
Owner Author

Owez commented Jan 2, 2023

You can now use --skip-metadata and --skip-download to do either option, I implemented the future issue in here

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
No open projects
Status: Done
Development

Successfully merging a pull request may close this issue.

1 participant