Skip to content

Conversation

@aitjcize
Copy link
Contributor

python 3.7 was deprecated. Update the version list to new python versions.

python 3.7 was deprecated. Update the version list to new python
versions.
@aitjcize
Copy link
Contributor Author

I'll fix the CI before reopening the PR.

@aitjcize aitjcize closed this Apr 11, 2025
@aitjcize aitjcize reopened this Apr 11, 2025
@aitjcize aitjcize changed the title github workflow: uprev python version github workflow: uprev python version and fix tests Apr 11, 2025
@aitjcize aitjcize force-pushed the main branch 3 times, most recently from e88445d to 9fba3e1 Compare April 11, 2025 07:15
@codecov-commenter
Copy link

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

Thanks for integrating Codecov - We've got you covered ☂️

@aitjcize
Copy link
Contributor Author

I've removed the test_sanity.py test, and instead read requirements*.txt from setup.py to avoid the need to maintain synchronization between setup.py and requirements*.txt. Thus the sanity test is no longer needed.

Migrate away from setuptools, which is deprecated from python 3.12+
Package requirements*.txt into the tarball, and directly read
requirements in setup.py
Copy link
Owner

@qqaatw qqaatw left a comment

Choose a reason for hiding this comment

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

Looks good - Thank you.

@qqaatw qqaatw merged commit 3114dd1 into qqaatw:main Apr 12, 2025
8 checks passed
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.

3 participants