This repository has been archived by the owner on Apr 26, 2024. It is now read-only.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Drop support for and remove references to EOL Python 3.6 #11683
Drop support for and remove references to EOL Python 3.6 #11683
Changes from all commits
caf0256
0b34cde
268b817
acef726
0add262
1b98edb
52bd94f
c74dcea
d717f1b
b979f33
dd20fc1
ebb53f4
bb476d4
7496e37
b156635
1ccaa28
c5e6dfa
9978524
b38d9ea
f810705
36f182f
0a566a9
112ee00
efef6d4
c5a9ad2
b76aaae
26d41fa
67a8d7b
8b695ef
954665d
cb763cc
7aaa772
8d209b2
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
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 we need to still check the platform implementation is
CPython
here -- I think pypy doesn't have these APIs.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.
pypy has
gc.collect
but notgc.freeze
In the interest of duck typing, we should probably check
hasattr(gc, 'freeze')
instead of examiningpython_implementation()
(Or we could say we're standardizing on CPython and rip the Pypy bits out of our codebase; no shame in that)
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.
Either of those seems ok, just a shame to delete something that's working.
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.
Great, I will re-implement the relevant parts of this check. Since this is already merged, what would be the best way of going about that? Opening a new PR restoring that functionality?
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.
Exactly! It probably makes sense to just use the same changelog entry so they'll get merged by towncrier.