-
Notifications
You must be signed in to change notification settings - Fork 67
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
add citation.cff file #434
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## development #434 +/- ##
============================================
Coverage 96.10% 96.10%
============================================
Files 11 11
Lines 2209 2209
============================================
Hits 2123 2123
Misses 86 86 ☔ 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.
In order to keep maintenance to a minimum, thoughts on removing version number and release date? I am ambivalent about it. If we leave it the same we should consider adding this to the PR checklist so that it doesn't get forgotten with new releases.
Agreed. I like to keep it simple and removed the version and release date. I've also added Quyen and Noah and updated the reference in the readme. |
Looks like scikit-learn just had a new release (https://scikit-learn.org/stable/whats_new/v1.6.html#version-1-6-0) and introduced a new tag |
Let's use a "pending" section in the change log since 3.0.0b0 is already out. |
Done. |
__init__.py