-
Notifications
You must be signed in to change notification settings - Fork 26
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
move build configuration into pyproject.toml
#216
move build configuration into pyproject.toml
#216
Conversation
converting to draft to work on building C extension under PEP517; should be tested with #218 |
588576d
to
f3c79f2
Compare
Codecov ReportBase: 87.17% // Head: 87.17% // No change to project coverage 👍
Additional details and impacted files@@ Coverage Diff @@
## master #216 +/- ##
=======================================
Coverage 87.17% 87.17%
=======================================
Files 17 17
Lines 1333 1333
=======================================
Hits 1162 1162
Misses 171 171
Flags with carried forward coverage won't be shown. Click here to find out more. Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
6e59c3e
to
6f630c3
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As far as I can tell, this looks good to me
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fix for dependencies
49b1e7c
to
91b0798
Compare
temporarily reverting this PR due to suspected involvement in https://jira.stsci.edu/browse/SPB-1432 |
This reverts commit 9af631f.
in line with PEP621, with the new
setuptools
; this also future-proofs against uses of a PEP517 build like in the new Tox update