From cfbde55e052092219c446dd078d8230b11372120 Mon Sep 17 00:00:00 2001 From: Ryan May Date: Thu, 31 Oct 2024 17:02:37 -0600 Subject: [PATCH] CI: Move --mpl to default pytest args in run-tests This eliminates the pytest-mpl piece being a mandatory part of the action. --- .github/actions/run-tests/action.yml | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/.github/actions/run-tests/action.yml b/.github/actions/run-tests/action.yml index 0098a14d04..3f3100faba 100644 --- a/.github/actions/run-tests/action.yml +++ b/.github/actions/run-tests/action.yml @@ -15,7 +15,7 @@ inputs: pytest-args: description: 'Additional arguments to pass to pytest' required: false - default: '-W error::metpy.deprecation.MetpyDeprecationWarning' + default: '--mpl -W error::metpy.deprecation.MetpyDeprecationWarning' runs: using: composite steps: @@ -30,7 +30,7 @@ runs: # By running coverage in "parallel" mode and "combining", we can clean up the path names run: | set -e -o pipefail - python -m coverage run -p -m pytest --mpl ${{ inputs.pytest-args }} tests/ 2>&1 | tee tests-${{ inputs.key }}.log + python -m coverage run -p -m pytest ${{ inputs.pytest-args }} tests/ 2>&1 | tee tests-${{ inputs.key }}.log python -m coverage combine python -m coverage report python -m coverage xml