-
Notifications
You must be signed in to change notification settings - Fork 36
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
Update Readme with Citation section #359
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #359 +/- ##
=======================================
Coverage 95.57% 95.57%
=======================================
Files 56 56
Lines 3138 3138
=======================================
Hits 2999 2999
Misses 139 139
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ 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.
@axiomcura This new citation should be updated in the CITATION.cff file as well.
Thanks @kenibrewer! I have updated the |
Co-authored-by: Gregory Way <gregory.way@gmail.com>
@d33bs @gwaybio @kenibrewer Hello! I have attended to all your comments and suggestions! |
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.
A few more minor fixes, but otherwise things LGTM
Co-authored-by: Ken Brewer <kenibrewer@users.noreply.github.com>
Co-authored-by: Ken Brewer <kenibrewer@users.noreply.github.com>
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.
Thanks Erik! I'll merge now :)
modified from EmbeddedArtistry
Description
Thank you for your contribution to pycytominer!
Please succinctly summarize your proposed change.
What motivated you to make this change?
This pull request introduces a section in the README titled
Citing pycytominer
. The changes made here will also reflect on the Read the Docs webpage, here is a preview, providing information on how to cite thepycytominer
package.Please also link to any relevant issues that your code is associated with.
What is the nature of your change?
Checklist
Please ensure that all boxes are checked before indicating that a pull request is ready for review.