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

Add CITATION.cff #7

Merged
merged 6 commits into from
Jun 22, 2022
Merged

Add CITATION.cff #7

merged 6 commits into from
Jun 22, 2022

Conversation

phackstock
Copy link
Contributor

Adding a CITATION.cff file so that it will be picked up by Zenodo for future releases.
For v0.1.0 I'll manually add the authors.
The authors are taken from setup.cfg.

@phackstock phackstock requested review from znicholls and jkikstra June 9, 2022 13:58
@phackstock phackstock self-assigned this Jun 9, 2022
Copy link
Collaborator

@znicholls znicholls left a comment

Choose a reason for hiding this comment

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

Should we also include the paper citation? I've never really worked out how to get the repo vs. paper citation thing work nicely. Maybe you have good examples from other stuff

@jkikstra
Copy link
Collaborator

jkikstra commented Jun 9, 2022

Paper citation is included in docs already, and probably should be linked on the Zenodo as well indeed.

Otherwise lgtm.

@znicholls
Copy link
Collaborator

I think you can do multiple references somehow in the citation file?

@phackstock
Copy link
Contributor Author

Ah true, I'll add some more info then.

@phackstock
Copy link
Contributor Author

I added some more info about the license and the DOIs from zenodo (both the version specific and the overall one)
@jkikstra, does the paper have a DOI yet? If so we could add it as well. If not I'd merge this PR and then work towards release 0.1.1 after we've merged #6.

@phackstock phackstock merged commit ba204db into iiasa:main Jun 22, 2022
@phackstock phackstock deleted the feature/add-citation branch June 22, 2022 07:45
@jkikstra
Copy link
Collaborator

Thanks. No DOI yet.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants