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

Fixed RegexMatchError #1200

Merged
merged 1 commit into from
Feb 4, 2022
Merged

Fixed RegexMatchError #1200

merged 1 commit into from
Feb 4, 2022

Conversation

DJStompZone
Copy link
Contributor

Fixed pytube.exceptions.RegexMatchError: __init__: could not find match for ^\w+\W

Solution proposed by juanchosaravia here
#1199 (comment)

@boardkeystown
Copy link

LETS GOOOOO!!!!! Let's get this pull request done so we can get pytube fixed and I can get it through pip again

@lemurmad
Copy link

Thanks for sending a fix at a fast pace. I was looking to see if anybody had fixed it.
Thanks again.

@RONNCC
Copy link
Collaborator

RONNCC commented Feb 4, 2022

Hey @DJStompZone
thanks for the PR -- can you add a unit test to show the behavior before and after and then I can merge :-)

Ideally this would 'break' in the future when youtube also changes behavior again and we can show it as a regression

@nficano nficano merged commit f23b20c into pytube:master Feb 4, 2022
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.

None yet

5 participants