-
Notifications
You must be signed in to change notification settings - Fork 95
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
[DOC] Add JOSS badges #815
Conversation
|
Do we need to pin a particular version? |
I think the current PR installed Line 48 in 4d5c645
might fix it. Not sure why it downgraded---for some reason, the most recent version of flake8 wants to use an older version of flake8-isort. For reference, the logs of the environment are linked: [this PR] [main] EDIT: Issue 1409 on flake8's repo has some discussion on this. Looks like pip has trouble resolving the correct versions. |
The upstream issue should now be fixed, so re-running the build might fix it---unless if circle-ci decides to re-use the old cache, in which case, any edit to |
Codecov Report
@@ Coverage Diff @@
## main #815 +/- ##
=======================================
Coverage 93.27% 93.27%
=======================================
Files 27 27
Lines 2215 2217 +2
=======================================
+ Hits 2066 2068 +2
Misses 149 149
Continue to review full report at Codecov.
|
Thanks @notZaki! It passed. |
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, thanks @tsalo !
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.
We should add:
- duecredit
- citation on README
- citation on index.rst
- references in workflow
@jbteves okay I think I got them all. I also cited the paper in the workflow boilerplate. |
Right now the first reference is numbered and subsequent ones are not. I lean towards asking that all of them be numbered rather than asking you to remove the first one's number, but I could go either way. |
Hm.. it wasn't numbered when I checked, but I guess that's because the Zenodo website was down at the time so the function must not have been working. I'll change it back. |
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.
This LGTM ! TY, @tsalo !! 📖
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, thanks @tsalo !
Closes #669.
Changes proposed in this pull request: