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

[FEATURE] Add output_options.keep_max_files to limit number of downloads per subscription #820

Merged
merged 5 commits into from
Nov 17, 2023

Conversation

jmbannon
Copy link
Owner

@jmbannon jmbannon commented Nov 17, 2023

Adds a new mechanism to limit subscriptions' number of videos to an explicit number (only supported date ranges prior to this).

Example usage:

output_options:
  keep_max_files: 10

will only keep 10 videos at max.

In addition:

  • keep_max_files is bundled into the Only Recent preset and can be used via setting the only_recent_max_files override variable (see examples or README for usage).
  • Changed the date_range variable name to only_recent_date_range for more clarity. Usage of date_range will continue to work.

Copy link

codecov bot commented Nov 17, 2023

Codecov Report

Attention: 2 lines in your changes are missing coverage. Please review.

Comparison is base (52d1a58) 96.81% compared to head (240fb7e) 96.78%.

Files Patch % Lines
...ytdl_sub/validators/string_formatter_validators.py 77.77% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #820      +/-   ##
==========================================
- Coverage   96.81%   96.78%   -0.03%     
==========================================
  Files          79       79              
  Lines        5114     5140      +26     
==========================================
+ Hits         4951     4975      +24     
- Misses        163      165       +2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@jmbannon jmbannon merged commit 544af22 into master Nov 17, 2023
35 checks passed
@jmbannon jmbannon deleted the j/keep_max_files branch November 17, 2023 17:01
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.

1 participant