Closed dustinswales closed 2 years ago
Looks good to me, although it looks like we need the scm/src/scm_output.F90 changes too: https://github.com/dustinswales/ccpp-scm/pull/2
I'll approve and merge once https://github.com/dustinswales/ccpp-scm/pull/2 is merged.
I'll approve and merge once dustinswales#2 is merged.
@grantfirl Thanks for your review. I've merged your PR in.
Looks good to me, although it looks like we need the scm/src/scm_output.F90 changes too: https://github.com/dustinswales/ccpp-scm/pull/2