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 up_deps.jl script to automate/streamline updating dependencies #409

Closed
Tracked by #358
valeriabarra opened this issue Aug 22, 2023 · 2 comments · Fixed by #410
Closed
Tracked by #358

Add up_deps.jl script to automate/streamline updating dependencies #409

valeriabarra opened this issue Aug 22, 2023 · 2 comments · Fixed by #410
Assignees
Labels
Dependencies 🔗 🍃 leaf Issue coupled to a PR

Comments

@valeriabarra
Copy link
Member

We should have an up_deps.jl similar to the one in ClimaAtmos and other CliMA repos (see, e.g., up_deps.jl), so that we can streamline and automate the update of dependencies instead of doing it manually, which is more error prone and tedious.

@LenkaNovak
Copy link
Collaborator

Thanks, @valeriabarra! This is related to the software release issue #358 so I'll tag it there. Note that some of the older examples (e.g. sea-breeze) are dependent on older packages, so I would only automate updating the Project only for the AMIP/ case in the experiments/ folder.

@LenkaNovak LenkaNovak added the 🍃 leaf Issue coupled to a PR label Aug 22, 2023
@valeriabarra valeriabarra mentioned this issue Aug 22, 2023
2 tasks
bors bot added a commit that referenced this issue Aug 22, 2023
410: Add up_deps.jl script r=valeriabarra a=valeriabarra

## Purpose 
The purpose of this PR is to add an `up_deps.jl` script to streamline/automate updating dependencies.

Closes #409 

## Content
- Solution implemented: 
    - [x] Added an `up_deps.jl` script that automatically updates the relevant environments

Review checklist

I have:
- followed the codebase contribution guide: https://clima.github.io/ClimateMachine.jl/latest/Contributing/
- followed the style guide: https://clima.github.io/ClimateMachine.jl/latest/DevDocs/CodeStyle/
- followed the documentation policy: https://github.com/CliMA/policies/wiki/Documentation-Policy
- checked that this PR does not duplicate an open PR.

In the Content, I have included 
- relevant unit tests, and integration tests, 
- appropriate docstrings on all functions, structs, and modules, and included relevant documentation.

----
- [x] I have read and checked the items on the review checklist.


Co-authored-by: Valeria Barra <[email protected]>
@valeriabarra
Copy link
Member Author

bors r+

bors bot added a commit that referenced this issue Aug 23, 2023
410: Add up_deps.jl script r=valeriabarra a=valeriabarra

## Purpose 
The purpose of this PR is to add an `up_deps.jl` script to streamline/automate updating dependencies.

Closes #409 

## Content
- Solution implemented: 
    - [x] Added an `up_deps.jl` script that automatically updates the relevant environments

Review checklist

I have:
- followed the codebase contribution guide: https://clima.github.io/ClimateMachine.jl/latest/Contributing/
- followed the style guide: https://clima.github.io/ClimateMachine.jl/latest/DevDocs/CodeStyle/
- followed the documentation policy: https://github.com/CliMA/policies/wiki/Documentation-Policy
- checked that this PR does not duplicate an open PR.

In the Content, I have included 
- relevant unit tests, and integration tests, 
- appropriate docstrings on all functions, structs, and modules, and included relevant documentation.

----
- [x] I have read and checked the items on the review checklist.


Co-authored-by: Valeria Barra <[email protected]>
bors bot added a commit that referenced this issue Aug 24, 2023
410: Add up_deps.jl script r=valeriabarra a=valeriabarra

## Purpose 
The purpose of this PR is to add an `up_deps.jl` script to streamline/automate updating dependencies. To update dependencies in the relevant environments, from now on, just simply run the following command:

```
julia ./.dev/up_deps.jl
```

Closes #409 

## Content
- Solution implemented: 
    - [x] Added an `up_deps.jl` script that automatically updates the relevant environments

Review checklist

I have:
- followed the codebase contribution guide: https://clima.github.io/ClimateMachine.jl/latest/Contributing/
- followed the style guide: https://clima.github.io/ClimateMachine.jl/latest/DevDocs/CodeStyle/
- followed the documentation policy: https://github.com/CliMA/policies/wiki/Documentation-Policy
- checked that this PR does not duplicate an open PR.

In the Content, I have included 
- relevant unit tests, and integration tests, 
- appropriate docstrings on all functions, structs, and modules, and included relevant documentation.

----
- [x] I have read and checked the items on the review checklist.


Co-authored-by: Valeria Barra <[email protected]>
@bors bors bot closed this as completed in b3650ff Aug 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Dependencies 🔗 🍃 leaf Issue coupled to a PR
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants