Skip to content

fix for youtube.py#245

Merged
daboross merged 1 commit intomasterfrom
unknown repository
Dec 10, 2018
Merged

fix for youtube.py#245
daboross merged 1 commit intomasterfrom
unknown repository

Conversation

@ghost
Copy link
Copy Markdown

@ghost ghost commented Aug 29, 2016

Implemented simple try/except blocks for a few variables in youtube.py -- catches the raised exception and still prints the title but with N/A as like/dislike/percentage.

@ghost ghost changed the title fix youtube.py fix for youtube.py Aug 29, 2016
@daboross
Copy link
Copy Markdown
Contributor

daboross commented Dec 8, 2016

I... guess this works. Ideally we would want to check if the optional things are in the result with the in operator, rather than catching an error, but if no one else objects I can merge this for now in a bit.

linuxdaemon pushed a commit to linuxdaemon/CloudBot that referenced this pull request Jun 30, 2018
Copy link
Copy Markdown
Contributor

@daboross daboross left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've tested this and it has no ill effects.

@daboross daboross merged commit e944377 into CloudBotIRC:master Dec 10, 2018
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