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

Update Manifest files #357

Merged
merged 1 commit into from
Jul 26, 2023
Merged

Update Manifest files #357

merged 1 commit into from
Jul 26, 2023

Conversation

valeriabarra
Copy link
Member

Purpose

This PR updates to the latest dependencies.

Closes #356

Content

  • Solution implemented: Updated Manifest.toml files using ] up

Review checklist

I have:

In the Content, I have included

  • relevant unit tests, and integration tests,
  • appropriate docstrings on all functions, structs, and modules, and included relevant documentation.

  • I have read and checked the items on the review checklist.

@valeriabarra
Copy link
Member Author

CI passes and this PR is ready for review.
Cc: @LenkaNovak

Copy link
Collaborator

@LenkaNovak LenkaNovak left a comment

Choose a reason for hiding this comment

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

LGTM! No behavioral changes from the previous merge! 🚀

@valeriabarra
Copy link
Member Author

LGTM! No behavioral changes from the previous merge! rocket

Great! I rebased and will merge

@valeriabarra
Copy link
Member Author

bors r+

bors bot added a commit that referenced this pull request Jul 25, 2023
357: Update Manifest files r=valeriabarra a=valeriabarra

## Purpose 
This PR updates to the latest dependencies.

Closes #356 


## Content
- Solution implemented: Updated `Manifest.toml` files using `] up`

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 <valeriabarra21@gmail.com>
@bors
Copy link
Contributor

bors bot commented Jul 25, 2023

Build failed:

@valeriabarra
Copy link
Member Author

bors r+

@bors
Copy link
Contributor

bors bot commented Jul 26, 2023

Build succeeded!

The publicly hosted instance of bors-ng is deprecated and will go away soon.

If you want to self-host your own instance, instructions are here.
For more help, visit the forum.

If you want to switch to GitHub's built-in merge queue, visit their help page.

@bors bors bot merged commit 8ca7482 into main Jul 26, 2023
7 of 8 checks passed
@bors bors bot deleted the valeria/update-deps branch July 26, 2023 00:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update dependencies
2 participants