-
Notifications
You must be signed in to change notification settings - Fork 44
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 fully to pyproject.toml #1737
Conversation
e1ff868
to
c781b87
Compare
Codecov Report
@@ Coverage Diff @@
## master #1737 +/- ##
==========================================
- Coverage 90.87% 90.85% -0.03%
==========================================
Files 74 74
Lines 4615 4615
==========================================
- Hits 4194 4193 -1
- Misses 421 422 +1
Flags with carried forward coverage won't be shown. Click here to find out more.
|
a8b7c8e
to
a2bb439
Compare
8d3ccee
to
ead1584
Compare
ead1584
to
f885928
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.
I did not see anything strange in this PR, although it is also not really my expertise.
The added comma in test_utils.py seems unnecessary, but is not important enough to block this PR.
Indeed, tell that to |
This PR replaces
setup.py
andMANIFEST.in
with a modern pyproject.toml file, as well as making many assorted tweaks to our CI and tests.