-
Notifications
You must be signed in to change notification settings - Fork 119
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
[production/AQM.v7] Enhancing EE2 compliance for the AQMv7 package #1034
[production/AQM.v7] Enhancing EE2 compliance for the AQMv7 package #1034
Conversation
…gger time for 4 cycles
@chan-hoo it looks like checks are blocking merge here, do you normally bypass that requirement to merge? |
@BrianCurtis-NOAA, uhm... strange. I didn't change anything. I have no idea what happens. |
@BrianCurtis-NOAA, I have not met this error before. Do you know how to fix this? |
Looks like it's mandatory a codeowner reviews, i'll have to add myself to codeowners soon. Can you please @chan-hoo |
1308f93
into
ufs-community:production/AQM.v7
@BrianCurtis-NOAA and @chan-hoo, you should be safe to bypass branch protections for this failure. A new documentation test has been added. The current documentation subdirectory in develop is Please feel free to merge this PR. |
DESCRIPTION OF CHANGES:
Removed "nowarn" flags in the ufs-weather-model's makefile and fixed "warning" issues related to CCPP
Deleted unused scripts and utilities
Fixed the hardcoded issues with AQM_UTIL and UFS_UTILS
Type of change
TESTS CONDUCTED:
CHECKLIST
LABELS (optional):
A Code Manager needs to add the following labels to this PR: