Clean-up typing_extensions -- typing_extensions#932
Conversation
There was a problem hiding this comment.
Thanks!
You (sensibly) removed a few non-underscored, but also not exported via __all__ attributes. As this is technically an API change (even though it's not exported via __all__), could you add these removals to typing_extensions/CHANGELOG:
TypingMetaOLD_GENERICSSUBS_TREEHAVE_ANNOTATEDHAVE_PROTOCOLS
Also could you mention in the Changelog that starting with 4.0.0 support for Python 2 and <3.6 has been dropped?
A few more remarks below.
de58c57 to
5d75e5f
Compare
5d75e5f to
12ec8de
Compare
|
Sorry, forgot to add a changelog entry earlier -- have now done so. |
srittau
left a comment
There was a problem hiding this comment.
Thanks again. For future reference: Please don't force push as it makes it harder to review changes.
|
Sorry - I had to rebase onto |
|
Yes, we squash merge anyway, so rebasing should not be necessary. |
xref #892
@srittau
Actual
typing_extensionsPR. Unchanged from original.A