Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Create CITATION.cff #388

Merged
merged 17 commits into from
May 22, 2023
Merged

Create CITATION.cff #388

merged 17 commits into from
May 22, 2023

Conversation

jukent
Copy link
Contributor

@jukent jukent commented May 2, 2023 •
edited
Loading

Adding a citation file to be understood by Zenodo.

Zenodo's example:

cff-version: 1.1.0
message: "If you use this software, please cite it as below."
authors:

Closes #309
date-released: 2023-05-01

@jukent jukent requested a review from a team as a code owner May 2, 2023 17:16
@jukent jukent requested review from jnmorley and r-ford and removed request for a team May 2, 2023 17:16
@github-actions
Copy link

github-actions bot commented May 2, 2023 •
edited
Loading

👋 Thanks for opening this PR! The Cookbook will be automatically built with GitHub Actions. To see the status of your deployment, click below.
🔍 Git commit SHA: 0909022
✅ Deployment Preview URL: https://ProjectPythia.github.io/pythia-foundations/_preview/388

@jukent jukent mentioned this pull request May 2, 2023
Closed
@jukent
Copy link
Contributor Author

jukent commented May 2, 2023

pre-commit doesn't like the format that zenodo suggests.

r-ford
r-ford previously approved these changes May 4, 2023
View reviewed changes
brian-rose
brian-rose reviewed May 4, 2023
View reviewed changes
CITATION.cff Outdated Show resolved Hide resolved
brian-rose
brian-rose reviewed May 4, 2023
View reviewed changes
Copy link
Member

@brian-rose brian-rose left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jukent Yay, we have pleased the linter gods!

I suggested a few tweaks to my own author entry.

CITATION.cff Outdated Show resolved Hide resolved
CITATION.cff Outdated Show resolved Hide resolved
jukent and others added 2 commits May 4, 2023 14:16
@jukent @brian-rose
Update CITATION.cff
a83542e 
Co-authored-by: Brian Rose <brose@albany.edu>
@jukent @brian-rose
Update CITATION.cff
a36a7b5 
Co-authored-by: Brian Rose <brose@albany.edu>
@jukent
Copy link
Contributor Author

jukent commented May 4, 2023

@ProjectPythia/core can everyone either comment here or add a commit changing your ORCID?

@brian-rose
Copy link
Member

@ProjectPythia/core can everyone either comment here or add a commit changing your ORCID?

We can also call this out at the Pythia meeting on Monday, we'll probably get a few more that way!

r-ford
r-ford reviewed May 4, 2023
View reviewed changes
CITATION.cff Outdated Show resolved Hide resolved
@jukent @r-ford
Update CITATION.cff
5e51660 
Co-authored-by: Robert Ford <50349951+r-ford@users.noreply.github.com>
@brian-rose brian-rose mentioned this pull request May 5, 2023
Closed
@ktyle
Copy link
Contributor

ktyle commented May 5, 2023

My ORCID ID is 0000-0001-5249-9665

andersy005
andersy005 reviewed May 8, 2023
View reviewed changes
CITATION.cff Outdated Show resolved Hide resolved
mgrover1
mgrover1 reviewed May 8, 2023
View reviewed changes
CITATION.cff Outdated Show resolved Hide resolved
jukent and others added 3 commits May 8, 2023 12:16
@jukent @mgrover1
Update CITATION.cff
f8b9cf1 
Co-authored-by: Max Grover <mgroverwx@gmail.com>
@jukent @andersy005
Update CITATION.cff
0ffdda3 
Co-authored-by: Anderson Banihirwe <13301940+andersy005@users.noreply.github.com>
@jukent
Update CITATION.cff with ORCIDs
8299928
@jukent jukent mentioned this pull request May 8, 2023
Merged
@jukent @ejlevine
Updates from main (#394)
800f708 
* Mention that SST is called tos in the model (#390)

* fix broken link (#393)

---------

Co-authored-by: Eric Levine <ejlevine@pm.me>
@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.

Powered by ReviewNB

@jukent
Copy link
Contributor Author

jukent commented May 8, 2023

We're now just missing ORCID's from @lkailynncar @kmpaul and @jnmorley

@mgrover1
Copy link
Contributor

mgrover1 commented May 8, 2023

@jukent - I believe this is @kmpaul 's ORCID
https://orcid.org/0000-0001-8155-8038

@jukent
Copy link
Contributor Author

jukent commented May 8, 2023

Now we're just missing @lkailynncar and she said she'd get it to me by the end of the week.

@jukent jukent merged commit 96805d1 into main May 22, 2023
3 checks passed
@jukent jukent deleted the citation branch May 22, 2023 19:32
github-actions bot pushed a commit that referenced this pull request May 22, 2023
@jukent @github-actions
Delete preview for pull request \#388
9e971c1
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@brian-rose brian-rose brian-rose left review comments

@andersy005 andersy005 andersy005 left review comments

@r-ford r-ford r-ford left review comments

@mgrover1 mgrover1 mgrover1 approved these changes

@jnmorley jnmorley Awaiting requested review from jnmorley jnmorley is a code owner automatically assigned from ProjectPythia/education

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Foundations should be citable with a DOI
6 participants
@jukent @brian-rose @ktyle @mgrover1 @andersy005 @r-ford