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
More mypy, fix a couple of return types #1253
Conversation
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.
Just a few nits, in an otherwise excellent looking PR!
Co-Authored-By: Graham Christensen <graham@grahamc.com>
Co-Authored-By: Graham Christensen <graham@grahamc.com>
Co-Authored-By: Graham Christensen <graham@grahamc.com>
Co-Authored-By: Graham Christensen <graham@grahamc.com>
Co-Authored-By: Graham Christensen <graham@grahamc.com>
Co-Authored-By: Graham Christensen <graham@grahamc.com>
back out one silly None and narrow an Any to str
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 think I've addressed all comments
Thank you a lot! |
Do my commits look good to you? |
looks great! I like the set annotation! |
More mypy, fix a couple of return types * back out one silly None and narrow an Any to str * str to Optional[str] for restore id arg type * black * backend: encrypted_links_to: prove that it is a set * backend: extract_key_options: type ET.Element Co-authored-by: Graham Christensen <graham@grahamc.com>
By the way, this is how this PR did:
|
More mypy, fix a couple of return types * back out one silly None and narrow an Any to str * str to Optional[str] for restore id arg type * black * backend: encrypted_links_to: prove that it is a set * backend: extract_key_options: type ET.Element Co-authored-by: Graham Christensen <graham@grahamc.com>
Feel free to edit further before merging if you like.