-
Notifications
You must be signed in to change notification settings - Fork 5
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
Add key.delete() #27
Add key.delete() #27
Conversation
ee2094a
to
b14a914
Compare
def func(): | ||
'''The bit that we want to retry''' | ||
key.delete(headers=headers) | ||
return True |
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.
The other variants don't seem to return True
?
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.
I don't think they need to return anything (as long as they raise an exception on failure)
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.
done
Done with first pass |
b14a914
to
57e7429
Compare
PTAL |
Aside from the lingering question about |
57e7429
to
975ab4b
Compare
975ab4b
to
8a94857
Compare
LGTM |
No description provided.