ESCOMP / CMEPS

NUOPC Community Mediator for Earth Prediction Systems
https://escomp.github.io/CMEPS/
24 stars 79 forks source link

add werror to extbuild #333

Closed jedwards4b closed 1 year ago

jedwards4b commented 1 year ago

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:

jedwards4b commented 1 year ago

I will run cesm prealphas with this + CDEPS https://github.com/ESCOMP/CDEPS/pull/207

jedwards4b commented 1 year ago

@uturuncoglu could you also review this one?

uturuncoglu commented 1 year ago

@jedwards4b maybe I could test both of them in the same time.

jedwards4b commented 1 year ago

Yes - please do!

DeniseWorthen commented 1 year ago

@uturuncoglu What testing are you planning? I don't want to replicate anything you were going to do on UFS.

uturuncoglu commented 1 year ago

@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.

DeniseWorthen commented 1 year ago

@uturuncoglu Sure, will do.