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.

Sign up for GitHub

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

Jump to bottom

More protection against None #6597

Merged

ericholscher merged 1 commit into readthedocs:master from stsewd:more-none-protection

Feb 3, 2020

Member

stsewd commented Jan 28, 2020

If the api fails, self.project and self.version are going to be None
and error when logging to sentry


          More protection against None

8b5882a

If the api fails, self.project and self.version are going to be None
and error when logging to sentry

stsewd requested a review from a team

January 29, 2020 16:55

ericholscher approved these changes

View reviewed changes

ericholscher merged commit 336a3ff into readthedocs:master

stsewd deleted the more-none-protection branch

February 3, 2020 21:24

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment