-
Notifications
You must be signed in to change notification settings - Fork 159
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
Inline post Updates #880
Inline post Updates #880
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
pre-test ran ok on weather model side.
Is there a UPP PR associated with this, or just bumping the hash to an already merged UPP? |
@BrianCurtis-NOAA UPP develop branch continues to be updated with PRs tested and merged. This PR is intended to catch up latest commit: NOAA-EMC/UPP@ce5f3b1. |
Testing for 2489 has completed, please continue with the merge process, thank you. |
Description
(Instructions: this, and all subsequent sections of text should be removed and filled in as appropriate.)
This PR is for:
Issue(s) addressed
Link the issues to be closed with this PR, whether in this repository, or in another repository.
(Remember, issues should always be created before starting work on a PR branch!)
Testing
How were these changes tested?
What compilers / HPCs was it tested with?
Are the changes covered by regression tests? (If not, why? Do new tests need to be added?)
Have the ufs-weather-model regression test been run? On what platform?
Dependencies
If testing this branch requires non-default branches in other repositories, list them.
Those branches should have matching names (ideally)
Do PRs in upstream repositories need to be merged first?
If so add the "waiting for other repos" label and list the upstream PRs
Requirements before merging