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

Urge new contributors to update the .zenodo.json + stronger urge to avoid cosima_cookbook.getvar #496

Merged
merged 6 commits into from
Feb 27, 2025

Conversation

chrisb13
Copy link
Contributor

Following #472, small change to contributing section such that authors know they can also add themselves to .json file for zenodo.

Would be good to also update 1 and 2

Following [this discussion](COSIMA#472), small change to `contributing` section such that authors know they can also add themselves to .json file for zenodo. Would be good to also update [1](https://github.com/COSIMA/cosima-recipes/wiki/1.-Working-on-a-recipe) and [2](https://cosima-recipes.readthedocs.io/en/latest/contributing.html)
@chrisb13
Copy link
Contributor Author

chrisb13 commented Feb 17, 2025

@navidcy, here's the discussed PR.

@chrisb13 chrisb13 mentioned this pull request Feb 17, 2025
@chrisb13 chrisb13 requested a review from navidcy February 18, 2025 01:38
@chrisb13
Copy link
Contributor Author

Perhaps @navidcy is busy at the moment. @adele-morrison are you able to review / update / approve this?

@navidcy
Copy link
Collaborator

navidcy commented Feb 27, 2025

Perhaps @navidcy is busy at the moment. @adele-morrison are you able to review / update / approve this?

I was indeed on leave.

Thanks for this @chrisb13!

@navidcy navidcy changed the title Update README.md Urge new contributors to update the .zenodo.json Feb 27, 2025
@navidcy
Copy link
Collaborator

navidcy commented Feb 27, 2025

hm... docs are failing... any ideas why?

@navidcy
Copy link
Collaborator

navidcy commented Feb 27, 2025

I can't seem to be able to retrigger it. @angus-g could you give me admin rights on the readthedocs for the repo?

@navidcy navidcy changed the title Urge new contributors to update the .zenodo.json Urge new contributors to update the .zenodo.json + stronger urge to avoid cosima_cookbook.getvar Feb 27, 2025
@navidcy
Copy link
Collaborator

navidcy commented Feb 27, 2025

thanks @angus-g !

@navidcy navidcy merged commit 261655c into COSIMA:main Feb 27, 2025
2 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.

4 participants