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

Fix seriesname typo from seriesnami #203

Merged
merged 1 commit into from
May 23, 2021
Merged

Conversation

kaelspencer
Copy link

Simple typo fix for self.seriesname. This caused seriesname to be valid for formatting the file name but it had a stale value.

#202

Simple typo fix for self.seriesname. This caused seriesname to be valid for formatting the file name but it had a stale value.
@codecov
Copy link

codecov bot commented May 23, 2021

Codecov Report

Merging #203 (702685e) into maint-3.0 (ba7e590) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           maint-3.0     #203   +/-   ##
==========================================
  Coverage      76.81%   76.81%           
==========================================
  Files             11       11           
  Lines            966      966           
  Branches         216      216           
==========================================
  Hits             742      742           
  Misses           153      153           
  Partials          71       71           
Impacted Files Coverage Δ
tvnamer/__init__.py 100.00% <100.00%> (ø)
tvnamer/utils.py 79.33% <100.00%> (ø)

@dbr dbr merged commit 2194acf into dbr:maint-3.0 May 23, 2021
@dbr
Copy link
Owner

dbr commented May 23, 2021

Thanks for this! Merged and shall make a new patch release momentarily

I was confused why the tests didn't catch this.. but.. they did! The checks for e673514 failed, I just didn't notice (muddied up in failing/passing-again notifications for the non-maint branch)

@dbr
Copy link
Owner

dbr commented May 23, 2021

Now released as tvnamer==3.0.4!

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