search

CABLE-LSM / benchcab

Tool for evaluation of CABLE land surface model
https://benchcab.readthedocs.io/en/latest/
Apache License 2.0
2 stars 3 forks source link

Added set -ex to the jobscript rendering, removed if statements. Fixe… #168

Closed bschroeter closed 11 months ago

bschroeter commented 11 months ago

Added set -ex to the header of the rendered jobscript. -x is used in place of explicit error messages as per the removed if statements, as it outputs all commands as they are being called so that we can see the exact line that the script fails.

In resolving this, the tests appear to be a bit flakey, relying on specific whitespace and line endings. I think this will be resolved as we move to Jinja2 templating (#142)

Fixes #158.

codecov[bot] commented 11 months ago

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@162f554). Click here to learn what that means. The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #168   +/-   ##
=======================================
  Coverage        ?   88.77%           
=======================================
  Files           ?       27           
  Lines           ?     1630           
  Branches        ?        0           
=======================================
  Hits            ?     1447           
  Misses          ?      183           
  Partials        ?        0