-
Notifications
You must be signed in to change notification settings - Fork 109
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
Hotfix issue 153 #161
Hotfix issue 153 #161
Conversation
455147d
to
856adfb
Compare
856adfb
to
6503ba7
Compare
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.
Thanks! LGTM, This handles the condition and stops us crashing. I can't see a better way to do this. If anyone else can we can forward fix!
Thank you for merging ! |
Yeah, I have it all automated apart from changes in README.rst - So releasing is easy here. Just need to not let it sit so long so we ship more with less change. |
This is an attempt to fix #153
This is my very first experience with
ast
and I'll admit I don't really know what I'm doing. I added a test based on the issue's description, checked that it failed with the current state of the code base as reported, and applied a minimal patch to make it pass without breaking any other test.