search

ESCOMP / CAM-SIMA

Community Atmosphere Model - System for Integrated Modeling of the Atmosphere
3 stars 12 forks source link

Allow python unit tests to automatically run after PR merge #185

Closed nusbaume closed 2 years ago

nusbaume commented 2 years ago

This pull request splits the Github Actions PR workflow into two separate workflows, one for unit tests and another for pylint. This allow for the unit tests to be run one final time after a PR has been merged, to ensure that the code still works as expected. There are also some minor code modifications found by pylint during testing.

Finally, after this PR has been merged I'll add a label to the README file in the main branch that will notify users as to whether the tests are currently passing (or not).

Fixes #180

nusbaume commented 2 years ago

@cacraigucar I added you as reviewer just in case, but if you feel like this PR is currently outside of your element then feel free to take yourself off. Thanks!

nusbaume commented 2 years ago

Hi all, just adding an FYI that the actions/setup-python calls in the CAMDEN workflows were out-of-date, so they have now been updated from v1 to v4 in the latest commit.