-
Notifications
You must be signed in to change notification settings - Fork 354
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: update PyPi metadata #8971
Conversation
✅ Deploy Preview for determined-ui canceled.
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #8971 +/- ##
==========================================
+ Coverage 47.34% 47.36% +0.01%
==========================================
Files 1162 1162
Lines 176133 176133
Branches 2237 2237
==========================================
+ Hits 83396 83422 +26
+ Misses 92579 92553 -26
Partials 158 158
Flags with carried forward coverage won't be shown. Click here to find out more. |
Failing Flake CI...
Fixed |
64e949a
to
4af4dfc
Compare
license="Apache License 2.0", | ||
classifiers=["License :: OSI Approved :: Apache Software License"], | ||
# Use find_namespace_packages because it will include data-only packages (that is, directories | ||
# containing only non-python files, like our gcp terraform directory). | ||
packages=setuptools.find_namespace_packages(include=["determined*"]), | ||
# Technically, we haven't supported 3.6 or tested against it since it went EOL. But some users | ||
# are still using it successfully so there's hardly a point in breaking them. | ||
python_requires=">=3.6", | ||
python_requires=">=3.8", |
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.
Updating to 3.7 can be done without checking with anyone (it already needs at least 3.7 to run). I'd like it if we could say 3.8, since that's the oldest version we test against, but I think we should check with Product and someone who can represent current customers to see if it'll break anyone's workflow.
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'm going to assume that our release tools will update the version to next-release. I'll keep an eye on it.
This is great. Thank you!
Description
Updates determined pip package metadata in PyPi.
DET-10187
Test Plan
Unsure how to test without deploying a new version.