-
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
Enable Dependabot updates to Poetry, fix tests and docs build #6
Conversation
Co-Authored-By: Jenna Tomkinson <[email protected]>
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! Excited to see the progress of this work!
tests/data/cytotable/NF1_cellpainting_data_shrunken/NF1_plate2_export_masks.cppipe
Outdated
Show resolved
Hide resolved
Thanks @jenna-tomkinson for the review here! I'm going to interpret your review as an approval for #4, as it goes through code which this branch was based on. I'll merge #4 and then this PR to keep the git history clean and descriptive. Related issues were created or referenced to help make sure we address what you mentioned in the comments. |
Added changes which are more related to #4 which weren't apparent until after a merge (automated tests were prevented and a docs build failed after merge). I'm trying to keep things succinct by appending the changes here. |
This PR focuses on long-term sustainability for this project by enabling Dependabot to automatically update the Poetry lockfile. Normally this is a task which must be performed manually. By enabling this change we can have Dependabot automatically update the dependency references (which still fall into the constraints of the
pyproject.toml
file).Thanks for any feedback!
Should be merged after #4