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] Create output_directory_nfo_tags first instead of last #809

Merged
merged 1 commit into from
Nov 6, 2023

Conversation

jmbannon
Copy link
Owner

@jmbannon jmbannon commented Nov 6, 2023

In addition, do not write empty tags

Copy link

codecov bot commented Nov 6, 2023

Codecov Report

Merging #809 (ea5d1b3) into master (e257210) will decrease coverage by 0.02%.
The diff coverage is 90.00%.

@@            Coverage Diff             @@
##           master     #809      +/-   ##
==========================================
- Coverage   96.82%   96.81%   -0.02%     
==========================================
  Files          79       79              
  Lines        5110     5111       +1     
==========================================
  Hits         4948     4948              
- Misses        162      163       +1     
Files Coverage Δ
src/ytdl_sub/plugins/output_directory_nfo_tags.py 100.00% <100.00%> (ø)
src/ytdl_sub/plugins/nfo_tags.py 96.77% <85.71%> (-0.98%) ⬇️

@jmbannon jmbannon merged commit 395691f into master Nov 6, 2023
35 checks passed
@jmbannon jmbannon deleted the j/nfo-update branch November 6, 2023 07:10
@jmbannon jmbannon mentioned this pull request Nov 7, 2023
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