Add CITATION.cff - #95
Merged
Merged
Conversation
mhweber
requested changes
Sep 8, 2026
mhweber
left a comment
mhweber
left a comment
Collaborator
There was a problem hiding this comment.
Dropped 'A.' from 'Ryan A.' for given name - assuming that given names should match others and be first name only and not include middle initial. Also, the PR request is for master branch and I have the paper in a paper branch so assuming this PR should be for the paper branch not master. Thank you!
goelakash
marked this pull request as ready for review
September 9, 2026 06:26
Author
|
@mhweber thank you for the review, i've applied the changes. |
goelakash
force-pushed
the
docs/add-citation-cff
branch
from
September 9, 2026 06:34
a40236d to
5a25c7c
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
JOSS reviewers ask for a
CITATION.cff, and this repository does not have one yet.Generated from the author list in
origin/Paper:inst/extdata/JOSS/paper.md: 5 author(s), with given and family names taken from each author's ORCID record rather than split from the display name, because compound surnames do not split reliably.3 author(s) have no ORCID in the paper, so their names were split at the last word. Please correct those if I put the boundary in the wrong place.
licenseisMIT, read from the LICENSE file.versionis0.11.0, the newest release tag.Verified with
cffconvert --validate: valid against schema 1.2.0.