-
Notifications
You must be signed in to change notification settings - Fork 259
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
Migrate to PEP 517 build system #746
Conversation
Codecov Report
@@ Coverage Diff @@
## master #746 +/- ##
=======================================
Coverage 92.86% 92.86%
=======================================
Files 35 35
Lines 5533 5533
Branches 941 941
=======================================
Hits 5138 5138
Misses 307 307
Partials 88 88
Flags with carried forward coverage won't be shown. Click here to find out more. Continue to review full report at Codecov.
|
This comment was marked as outdated.
This comment was marked as outdated.
- all remaining setuptools configuration is moved from setup.py to setup.cfg - description is now checked with twine rather than `setup.py check` Split from #734, including suggestions from @webknjaz Co-authored-by: Sviatoslav Sydorenko <[email protected]>
d06e03e
to
7abaec6
Compare
This pull request fixes 1 alert when merging 7abaec6 into c8a4334 - view on LGTM.com fixed alerts:
|
What do these changes do?
Migrate to PEP 517 build system
setup.py check
Are there changes in behavior for the user?
no
Related issue number
split from #734
Checklist
CHANGES.txt