-
Notifications
You must be signed in to change notification settings - Fork 673
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
build: use pep517/pep621 #1140
base: master
Are you sure you want to change the base?
build: use pep517/pep621 #1140
Conversation
Thank you for modernizing the project @trim21! I've made a few fix to the CI runners, can you try rebasing your PR please? However, I do not explain the failure for Python 3.6. |
would you mind enable this option in repo settings? so I can update branch in web-ui instead of local git client. |
Done, thanks for the suggestion. 👍 |
I guess, ci failed because setuptools support python3.5/3.6 are very out-dated and doesn't support new feature we are using. |
at fitst I want to continue using setuptools but python3.5/3.6 are too old and setuptools support theee 2 versions missing some feature we need and doesn't have a full support for pep621. So I have to use flit to make it work on these 2 versions. If you want to continue using setuptools we need drop these 2 python versions first. |
replace legacy setup.py with pyproject.toml pep517/pep621 config