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

Can next set of PRs include a ccpp-framework update (CI updates only) #2216

Closed
climbfuji opened this issue Apr 1, 2024 · 1 comment
Closed
Labels
enhancement New feature or request

Comments

@climbfuji
Copy link
Collaborator

Description

This was merged without being tested in the UFS, and the submodule pointer needs to be updated: NCAR/ccpp-framework#551 (comment)

Solution

It only affects the CI scripts in ccpp-framework, but still it would be nice to include this update in one of the next PRs.

Alternatives

Wait until the next ccpp-framework PR is tested in the UFS WM

Related to

n/a

@DusanJovic-NOAA
Copy link
Collaborator

I updated ccpp/framework submodule (to 0f416a2 commit) in my branch for #2218 (remove nowarn) and submitted regression test on Hercules. So far ~20 tests passed. I expect all tests will pass, I do not see any issues with updated ccpp/framework.

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

No branches or pull requests

2 participants