-
-
Notifications
You must be signed in to change notification settings - Fork 76
Use CITATION.cff standard for citation information #484
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
base: main
Are you sure you want to change the base?
Conversation
🆗 Pre-flight checks passed 😃This pull request has been checked and contains no modified workflow files, spoofing, or invalid commits. Results of any additional workflows will appear here when they are done. |
|
Thanks for starting this Jaro. I noticed that we aren't linking properly to the citation file from the readme, so I've added a fix into this PR (rather than open a separate PR). My ORCID is: https://orcid.org/0000-0002-1248-3594 |
|
My ORCID is: https://orcid.org/0000-0001-8985-6814 |
|
I've now added my ORCID in a commit along with Doug's and Steve's |
|
@JaroCamphuijsen since it looks like all maintainers have added their details, can you confirm that we can remove WIP? I have looked through the changes, and they look fine to me. @anenadic was there any particular reason you requested review from @Devaraj-G or can we go ahead an merge? |
|
@bielsnohr I added a few more details, two names that were in the deleted AUTHORS file but weren't yet present in the CITATION.cff file and the orcids of my former colleagues. I think this is ready to go now! |
| author = {Aleksandra Nenadic and | ||
| Steve Crouch and | ||
| Thomas Kiley and | ||
| Matthew and |
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.
Is the family name missing or is it just Matthew?
Do not know which Mathew this refers to, either Matthew Bluteau or Matthew Graham per CITATION.cff or someone else.
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.
@anenadic This list of authors also has some other strange names (supposedly Github usernames) in there. Was this intentional? Is this what paper.bib should contain?
Change from CITATION.md to CITATION.cff and use the proper data standard for .cff files. This will also enable the CITATION.cff-github integration and show the citation button in the web UI.
Additionally removed the AUTHORS file in the root of the repository. It is only mentioned (and linked to wrongly) in the paper.md file and contains duplicate information that already diverged from the content of the original CITATION.md. I added the additional authors from the AUTHORS file in the new CITATION.cff file.
Currently, the authors in CITATION.cff only have a given name and family name. It would be good to add an ORCID for everyone, perhaps this is something to ask in the slack channel/mailinglist and/or through other channels? I'm happy to put them all in if the authors post their ORCID as a comment to this PR.
fixes #477