-
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
#56
move build configuration into pyproject.toml
#56
Conversation
317da7a
to
e81eeaa
Compare
Codecov ReportBase: 0.00% // Head: 0.00% // No change to project coverage 👍
Additional details and impacted files@@ Coverage Diff @@
## master #56 +/- ##
======================================
Coverage 0.00% 0.00%
======================================
Files 25 25
Lines 3060 3060
======================================
Misses 3060 3060 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. |
I was able to get the tests to run through correctly by removing usage of |
39cddf2
to
7d8d689
Compare
b5955d1
to
2680274
Compare
2680274
to
976b799
Compare
976b799
to
fa28697
Compare
fa28697
to
bb5311e
Compare
4b14a80
to
17b7c41
Compare
@WilliamJamieson I didn't realize that |
bae487e
to
6fca1e4
Compare
Co-authored-by: William Jamieson <wjamieson@stsci.edu>
Co-authored-by: William Jamieson <wjamieson@stsci.edu>
6fca1e4
to
4576e39
Compare
setuptools
now supports the[project]
table, which is defined by PEP621.Additionally,
setuptools
now supports its own entry inpyproject.toml
called[tool.setuptools]
(pypa/setuptools#1688, https://setuptools.pypa.io/en/latest/userguide/pyproject_config.html#setuptools-specific-configuration); however, it comes with the following disclaimer:Given this, we can attempt to consolidate the build configuration into a single
pyproject.toml
file that can possibly be read by other build systems in the future.