Skip to content

Conversation

@ptrcnull
Copy link
Collaborator

@ptrcnull ptrcnull commented Aug 2, 2024

with some types of episodes, such as extras, the episode number will be missing, but the subtitle download succeeds with the check removed

with some types of episodes, such as extras, the episode number
will be missing, but the subtitle download succeeds with the check removed
@ptrcnull ptrcnull requested a review from getzze August 2, 2024 14:28
@github-actions
Copy link
Contributor

github-actions bot commented Aug 2, 2024

Coverage report

Click to see where and how coverage changed

FileStatementsMissingCoverageCoverage
(new stmts)
Lines missing
  subliminal
  video.py
Project Total  

This report was generated by python-coverage-comment-action

@ptrcnull ptrcnull marked this pull request as draft August 2, 2024 14:32
@ptrcnull
Copy link
Collaborator Author

ptrcnull commented Aug 2, 2024

ah, actually, sometimes it just pulls the first episode of the season...

@getzze
Copy link
Collaborator

getzze commented Aug 2, 2024

Fixes #692

And it would be great if this PR could also tackle #678

They don't look easy to fix though...

@getzze
Copy link
Collaborator

getzze commented Sep 11, 2024

Are you still working on this?
I just realized it's a duplicate of: #702

@ptrcnull
Copy link
Collaborator Author

to be fair, the other MR is flawed in the same way as mine is

@ptrcnull
Copy link
Collaborator Author

ptrcnull commented Sep 11, 2024

but to answer your question, i forgot about it, but at some point i'd like to either fix it myself, or see it fixed ^^

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.

3 participants