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
hashes: Do not import str #2662
hashes: Do not import str #2662
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.
ACK 984e1a2 not API-visible, fine to do alongside the release
984e1a2
to
0ae750b
Compare
Depending on things being in scope for macros to use is bad form, using the fully qualified path is the correct way. Do not import `str` instead use the fully qualified path to the `core` re-export. Use fully qualified instead.
0ae750b
to
71bb862
Compare
This PR (combined with #2666) demonstrates the new branching strategy explained here: #2657 (comment) This PR targets |
Pull Request Test Coverage Report for Build 8574970411Details
💛 - Coveralls |
Candidate for merge now please, acked and open for over two weeks with no comments or nacks. |
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.
ACK 71bb862 trivial rebase
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.
ACK 71bb862
Depending on things being in scope for macros to use is bad form, using the fully qualified path is the correct way.
Do not import
str
instead use the fully qualified path to thecore
re-export.Use fully qualified path instead.