Skip to content

Adds CITATION file and edits README to include Zenodo badge#234

Merged
guillermoaguilar merged 5 commits intowichmann-lab:mainfrom
guillermoaguilar:citationfile
Jan 29, 2025
Merged

Adds CITATION file and edits README to include Zenodo badge#234
guillermoaguilar merged 5 commits intowichmann-lab:mainfrom
guillermoaguilar:citationfile

Conversation

@guillermoaguilar
Copy link
Collaborator

No description provided.

@guillermoaguilar guillermoaguilar changed the title Adds CITATION file Adds CITATION file and edits README to include Zenodo badge Jan 27, 2025
@otizonaizit
Copy link
Collaborator

Maybe we should add a "How to cite" paragraph to the entry page in the documentation? This would complement the DOI badge in the README file.

Copy link
Collaborator

@pberkes pberkes left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@otizonaizit otizonaizit self-requested a review January 28, 2025 15:47
Copy link
Collaborator

@otizonaizit otizonaizit left a comment

Choose a reason for hiding this comment

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

Cool! Could you add the same paragraph you added in the README.md also in docs/index.md (I guess at the end after Acknowledgements?)

@guillermoaguilar
Copy link
Collaborator Author

Done, I put it before the history in docs/index.md

Take a look: https://psignifit--234.org.readthedocs.build/en/234/

If everyone is OK with everything now, I'll officially release tomorrow.

Copy link
Collaborator

@otizonaizit otizonaizit left a comment

Choose a reason for hiding this comment

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

Perfect!

@pberkes
Copy link
Collaborator

pberkes commented Jan 29, 2025

Looking good, thank you @guillermoaguilar !

@guillermoaguilar guillermoaguilar merged commit d347790 into wichmann-lab:main Jan 29, 2025
8 checks passed
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