-
-
Notifications
You must be signed in to change notification settings - Fork 341
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 error handling #1347
Fix error handling #1347
Conversation
cd567c0
to
08b575f
Compare
08b575f
to
d7bd500
Compare
urgh, github closed my PR while I was fixing a dirty rebase... |
PTAL |
Seems like a known bug in pygithub: PyGithub/PyGithub#3179 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This fix is too generic. I'll push the correct code later.
Thanks, Matt. The upstream is working on a fix (PyGithub/PyGithub#3180) so I was hoping to quickly work around this and revert it once a new release is out. But if you have time to refine it I am also happy 🙏 |
Are you thinking of checking |
yes see #1348 |
Try this to see if #1346 can be fixed. I can reproduce the error locally and I noticed that despite it's a 404 error, pygithub does not raise
github.UnknownObjectException
as documented, but instead raises the basegithub.GithubtException
. Something might have changed in recent pygithub versions. I tested this with 2.4.0.Let's try to merge this and see if it fixes 1346.