-
-
Notifications
You must be signed in to change notification settings - Fork 8
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
chore: drop python 3.8 support and upgrade ETH dependencies #118
Conversation
related to the other discussion, I wonder if these lower-level packages than ape should stay 3.8 a little bit longer, enough time for those using ape to upgrade to 0.8 where py 3.8 is dropped then we drop this closer to the deprecation date (maybe when we go v1.0 with ape) |
I am going back and forth. The main thing is I need the eth dependencies upgraded and I can't do that without also it being breaking (think Hexbytes change). So I need breaking changes releases for all these lower levels packages to even upgrade ape. I guess a chicken and egg problem. |
@fubuloubu OK here is a separate PR that add 3.12, so now it will support 3.8-3.12 withou breaking #119 |
1569c5c
to
49236ff
Compare
49236ff
to
65ae7e9
Compare
@@ -1,4 +1,4 @@ | |||
from typing import Iterator | |||
from collections.abc import Iterator |
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.
what's the story behind this change? I saw this somewhere else and didn't get why
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.
typing
is deprecated as of 3.9
https://docs.python.org/3/library/typing.html#typing.Sequence
where it introduced generic types for collections like list
and dict
https://docs.python.org/3/library/stdtypes.html#types-genericalias
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.
typing has aliases to collections.abc but theyre deprecated. Pre python 3.9, they were required
bytes_val = ( | ||
self._selector_hash_fn(aitem.selector)[:4] | ||
if isinstance(aitem, (MethodABI, ErrorABI)) | ||
else self._selector_hash_fn(aitem.selector) |
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 of this one as the weird one (basically for EventABI
it only uses 32 bytes)
1246058
to
f0d0be2
Compare
What I did
fixes: #
How I did it
How to verify it
Checklist