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.

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

Try citation.cff #1767

Merged
merged 3 commits into from
May 17, 2024
Merged

Try citation.cff #1767

merged 3 commits into from
May 17, 2024

Conversation

dlakaplan
Copy link
Contributor

This can be updated. The paper citation and overall author list need not be the same.

@dlakaplan
Copy link
Contributor Author

Can also add the zenodo DOI based on fixed versions if/when we do that.

@dlakaplan
Copy link
Contributor Author

Should probably update the full author list if people think this is worth doing.

@dlakaplan
Copy link
Contributor Author

Although, this may not work for > 1 paper so it may not be workable.

@dlakaplan dlakaplan added awaiting review This PR needs someone to review it so it can be merged minor A minor PR that doesn't need a lot of thought labels May 16, 2024
@abhisrkckl
Copy link
Contributor

Although, this may not work for > 1 paper so it may not be workable.

Unfortunately, that doesn't work. According to the authors of cff, this is a feature.
citation-file-format/citation-file-format#339

@abhisrkckl
Copy link
Contributor

Should probably update the full author list if people think this is worth doing.

I think this is super helpful.

@abhisrkckl
Copy link
Contributor

Can also add the zenodo DOI based on fixed versions if/when we do that.

ApJ people asked me to upload the version corresponding to the paper to zenodo. So we'll have a new zenodo DOI soon.

@dlakaplan
Copy link
Contributor Author

I updated the author list to the full list of contributors, but kept the paper as-is. We'll see what we do when we have a second paper.

@dlakaplan
Copy link
Contributor Author

Can also add the zenodo DOI based on fixed versions if/when we do that.

ApJ people asked me to upload the version corresponding to the paper to zenodo. So we'll have a new zenodo DOI soon.

Yes, exactly.

citation.cff Outdated Show resolved Hide resolved
@abhisrkckl abhisrkckl merged commit 8246e7c into nanograv:master May 17, 2024
7 checks passed
@abhisrkckl abhisrkckl mentioned this pull request Jun 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
awaiting review This PR needs someone to review it so it can be merged minor A minor PR that doesn't need a lot of thought
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants