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

Ignore padding exceptions. Return remaining codes. #25

Merged
merged 1 commit into from
Sep 24, 2020

Conversation

EpicVoyage
Copy link
Contributor

No description provided.

@trafico-bot trafico-bot bot added the 🔍 Ready for Review Pull Request is not reviewed yet label Sep 24, 2020
@lgtm-com
Copy link

lgtm-com bot commented Sep 24, 2020

This pull request introduces 1 alert when merging 5805255 into 82d72e6 - view on LGTM.com

new alerts:

  • 1 for Except block handles 'BaseException'

@trafico-bot trafico-bot bot added ✅ Approved Pull Request has been approved and can be merged and removed 🔍 Ready for Review Pull Request is not reviewed yet labels Sep 24, 2020
@moul moul merged commit c158789 into moul:develop Sep 24, 2020
@trafico-bot trafico-bot bot added ✨ Merged Pull Request has been merged successfully and removed ✅ Approved Pull Request has been approved and can be merged labels Sep 24, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ Merged Pull Request has been merged successfully
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants