-
Notifications
You must be signed in to change notification settings - Fork 2.1k
[LABS-244] Remove python 3.9 support #20099
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
base: main
Are you sure you want to change the base?
Conversation
9090596
to
1b4ead5
Compare
sudo dnf update -y | ||
sudo dnf search python3.12 | ||
sudo dnf install python3.12 -y |
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.
maybe. but maybe we should be testing on a newer amazon?
what's the reason for yum vs dnf. also odd that we need to use sudo for one and not the other. was the search for debug?
files = [ | ||
{file = "zstd-1.5.5.1-cp310-cp310-macosx_11_0_arm64.whl", hash = "sha256:b5cd20afab8d13c52d2b2219bf18cc765eae87b8219343bce20647007890adab"}, | ||
{file = "zstd-1.5.5.1-cp310-cp310-macosx_11_0_x86_64.whl", hash = "sha256:0f467ab9b57ab8b4b874e6974d38b802f20406803bb7ec9308df923553cd48f7"}, | ||
{file = "zstd-1.5.5.1-cp310-cp310-macosx_13_0_arm64.whl", hash = "sha256:1866fd2eb37bc80cd0a1bab8f099df5e778a773e829378c633f9fa1325a5dae1"}, |
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.
something is weird here with these being removed. will chat directly about this.
All alerts resolved. Learn more about Socket for GitHub. This PR previously contained dependency changes with security issues that have been resolved, removed, or ignored. |
Big changes:
Union
typesare no longerwill soon not be supported through Ruff. UseA | B
orA | None
forOptional[A]
kw_only
is now a supported option across all versions for dataclasses! Probably a good idea to consider this as the default.