Skip to content

ci: Add Python 3.13 build #170

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

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

DiamondJoseph
Copy link

Add Python 3.13 to versions tested in CI to ensure compatibility with the Numpy deprecation policy

Add Python 3.13 to versions tested in CI to ensure compatibility with the Numpy deprecation policy
@@ -119,12 +119,12 @@ def get_numpy_include_dirs():
author='Michael Davidsaver',
author_email='[email protected]',
license='BSD',
license_files=["LICENSE"],
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Any chance that this change is related to the error I now see from GHA runs?

   python -m twine check dist/p4p*.whl
  shell: /usr/bin/bash -e {0}
...
Checking dist/p4p-4.2.0-cp36-cp36m-manylinux1_i686.whl: ERROR    InvalidDistribution: Invalid distribution metadata: unrecognized or    
         malformed field 'license-file'    

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like it's some combination of build versions, testing some fixes

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have pushed ab76714 to fix the issue with actions/upload-artifact@v4 name uniqueness.

Reading through this twine, it isn't clear to me if this issue with the packaging module is with the P4P build, or with twine check? If the later case, then it might be better to pull in a specific version when installing twine?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants