payu-org / payu

A workflow management tool for numerical models on the NCI computing systems
Apache License 2.0
18 stars 25 forks source link

Added mail on final run option + test #300

Open aidanheerdegen opened 3 years ago

aidanheerdegen commented 3 years ago

Closes #299

Adds a config option to mail user when last run of a sequence completes.

pep8speaks commented 3 years ago

Hello @aidanheerdegen! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

There are currently no PEP 8 issues detected in this Pull Request. Cheers! :beers:

Comment last updated at 2021-04-01 03:02:27 UTC
coveralls commented 3 years ago

Coverage Status

Coverage increased (+0.2%) to 42.337% when pulling 6f98ff3f81b9c7a1d2ebf838fde293b9b37d641d on issue-299 into a83d60b09f8ff93432cde302d36904edc4b79f16 on master.

navidcy commented 3 years ago

Can't really comment on the python implementation of this, but I applaud the feature.