-
Notifications
You must be signed in to change notification settings - Fork 1
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 to pyproject.toml
#3
Conversation
Welcome to Codecov 🎉Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests. Thanks for integrating Codecov - We've got you covered ☂️ |
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 guess at some point we'd want this repo to be more consistent with napari
testing setup than the brainglobe
testing setup (given that this might become a napari
thing one day)? But this will do for our development ease, for now.
Before submitting a pull request (PR), please read the contributing guide.
Please fill out as much of this template as you can, but if you have any problems or questions, just leave a comment and we will help out :)
Description
What is this PR
Why is this PR needed?
Remove some of our outdated config files and use
pyproject.toml
standard instead.What does this PR do?
pyproject.toml
instead of splitting it betweensetup.cfg
andtox.ini
, etc.References
How has this PR been tested?
Local
pip install -e .
behaves identically tomain
. (Note that the widget on main remains broken, but it has the same error.)Is this a breaking change?
Does this PR require an update to the documentation?
Checklist: