-
Notifications
You must be signed in to change notification settings - Fork 79
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 werror to extbuild #333
add werror to extbuild #333
Conversation
I will run cesm prealphas with this + CDEPS ESCOMP/CDEPS#207 |
@uturuncoglu could you also review this one? |
@jedwards4b maybe I could test both of them in the same time. |
Yes - please do! |
@uturuncoglu What testing are you planning? I don't want to replicate anything you were going to do on UFS. |
@DeniseWorthen Since i need to test CDEPS PR. I think that could run couple of coupled test but if you want to run the entire RT against this PR please go ahead and I could test CDEPS seperately. |
@uturuncoglu Sure, will do. |
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.
All UFS tests passed.
Description of changes
Add werror to github testing - no gfortran warnings allowed.
Specific notes
Contributors other than yourself, if any:
CMEPS Issues Fixed (include github issue #):
Are changes expected to change answers? (specify if bfb, different at roundoff, more substantial)
BFB
Any User Interface Changes (namelist or namelist defaults changes)? NO
Testing performed
Testing performed if application target is CESM:
Testing performed if application target is UFS-coupled:
Testing performed if application target is UFS-HAFS:
Hashes used for testing: