You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
As part of the testing workflow, we should run the Fortran (pfunit-based) unit tests on each PR. Ideally, most or all of this repository would be covered by unit tests, and the unit testing together with the existing scripts_regression_tests would be sufficient coverage for a CESM_share PR to be merged.
See #17 (review) for the current process, though #18 suggests how we could make this more straightforward – but I think from an automated testing perspective it might not make much difference.
Note that this will require downloading & building pfunit as part of the workflow (probably cached like we cache netcdf, etc.).
The text was updated successfully, but these errors were encountered:
As part of the testing workflow, we should run the Fortran (pfunit-based) unit tests on each PR. Ideally, most or all of this repository would be covered by unit tests, and the unit testing together with the existing scripts_regression_tests would be sufficient coverage for a CESM_share PR to be merged.
See #17 (review) for the current process, though #18 suggests how we could make this more straightforward – but I think from an automated testing perspective it might not make much difference.
Note that this will require downloading & building pfunit as part of the workflow (probably cached like we cache netcdf, etc.).
The text was updated successfully, but these errors were encountered: