-
Notifications
You must be signed in to change notification settings - Fork 2
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
Prepare for cellfinder
merge
#54
Conversation
d27d9c1
to
df16e45
Compare
With the removal of tox.ini from main branch, also remove the now-redundant check on the development version of cellfinder
df16e45
to
4a2c551
Compare
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #54 +/- ##
=======================================
Coverage 81.33% 81.33%
=======================================
Files 32 32
Lines 1586 1586
=======================================
Hits 1290 1290
Misses 296 296 ☔ View full report in Codecov by Sentry. |
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.
LGTM. I made some small changes to the readme and suggested another.
Co-authored-by: Adam Tyson <[email protected]>
Fab, will merge when https://github.com/brainglobe/brainglobe-workflows/actions/runs/7396094521 completes. (This is running on 036b083, which is the last edit to the actual code, and I don't fancy waiting another 4 hours for the tests to pass again). |
See brainglobe/BrainGlobe#46
Prepares the package for the update to
cellfinder
cellfinder_core
->cellfinder.core
everywherecellfinder
, if requestedcellfinder>=1.0.0
.napari
optional dependency is still available.cellfinder
is now calledbrainmapper
.On merge, create new released on PyPI