-
Notifications
You must be signed in to change notification settings - Fork 52
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
[MRG] DOC: add JOSS badge and CITATION.cff #700
Conversation
@jasmainak do you think we need a citation statement in the README as well, or is the badge and "Cite this repository" tab Github adds with CITATION.cff adequate? |
You won’t see the github tab on the website, so I think a redundancy in the
readme won’t hurt!
…Sent from my iPhone
On Tue 19 Dec 2023 at 14:17, Ryan Thorpe ***@***.***> wrote:
@jasmainak <https://github.com/jasmainak> do you think we need a citation
statement in the README as well, or is the badge and "Cite this repository"
tab Github adds with CITATION.cff adequate?
—
Reply to this email directly, view it on GitHub
<#700 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ADY6FIRTB7JAOJ5C5RXJSNLYKHR3DAVCNFSM6AAAAABA3VMW4WVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNRTGM2DCNJTGY>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Codecov ReportAttention:
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## master #700 +/- ##
=======================================
Coverage 91.34% 91.34%
=======================================
Files 25 25
Lines 4599 4599
=======================================
Hits 4201 4201
Misses 398 398 ☔ View full report in Codecov by Sentry. |
Here's how the citation section looks in the compiled README. If it looks good, feel free to merge @jasmainak. |
Looks good, thanks @rythorpe !! |
No description provided.