Skip to content
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

Merged
merged 1 commit into from
Apr 9, 2024
Merged

chore: update PyPi metadata #8971

merged 1 commit into from
Apr 9, 2024

Conversation

sirredbeard
Copy link
Member

@sirredbeard sirredbeard commented Mar 7, 2024

Description

Updates determined pip package metadata in PyPi.

  • adds content of README.md
  • updates e-mail address
  • updates Python to version 3.8+

DET-10187

Test Plan

Unsure how to test without deploying a new version.

@cla-bot cla-bot bot added the cla-signed label Mar 7, 2024
Copy link

netlify bot commented Mar 7, 2024

Deploy Preview for determined-ui canceled.

Name Link
🔨 Latest commit 4af4dfc
🔍 Latest deploy log https://app.netlify.com/sites/determined-ui/deploys/65e9e1547deef60008dc6413

Copy link

codecov bot commented Mar 7, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 47.36%. Comparing base (c7e5d43) to head (4af4dfc).
Report is 8 commits behind head on main.

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              
Flag Coverage Δ
backend 42.70% <ø> (+0.06%) ⬆️
harness 63.94% <ø> (ø)
web 42.54% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

see 5 files with indirect coverage changes

@sirredbeard
Copy link
Member Author

sirredbeard commented Mar 7, 2024

Failing Flake CI...

harness/setup.py:14:21: E251 unexpected spaces around keyword / parameter equals
    long_description = markdown_description,
                    ^
harness/setup.py:14:23: E251 unexpected spaces around keyword / parameter equals
    long_description = markdown_description,
                      ^
harness/setup.py:15:34: E251 unexpected spaces around keyword / parameter equals
    long_description_content_type = "text/markdown",
                                 ^
harness/setup.py:15:36: E251 unexpected spaces around keyword / parameter equals
    long_description_content_type = "text/markdown",

Fixed

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",
Copy link
Contributor

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.

@sirredbeard sirredbeard marked this pull request as ready for review April 9, 2024 19:52
@sirredbeard sirredbeard requested a review from a team as a code owner April 9, 2024 19:52
@sirredbeard sirredbeard enabled auto-merge (squash) April 9, 2024 19:53
Copy link
Contributor

@wes-turner wes-turner left a 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!

@sirredbeard sirredbeard merged commit f7a5260 into main Apr 9, 2024
72 of 84 checks passed
@sirredbeard sirredbeard deleted the update_pypi_listing branch April 9, 2024 19:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants