From f3f9a9f297419ca4dee28cc5b2a512fe070e7141 Mon Sep 17 00:00:00 2001 From: Jonathan Plasse <13716151+JonathanPlasse@users.noreply.github.com> Date: Fri, 31 Mar 2023 20:47:04 +0200 Subject: [PATCH] Fix pre-commit CI job exit code (#3833) --- .github/workflows/ci.yaml | 2 ++ scripts/pyproject.toml | 5 +++-- 2 files changed, 5 insertions(+), 2 deletions(-) diff --git a/.github/workflows/ci.yaml b/.github/workflows/ci.yaml index 0985896c6c..335f861988 100644 --- a/.github/workflows/ci.yaml +++ b/.github/workflows/ci.yaml @@ -227,4 +227,6 @@ jobs: # Enable color output for pre-commit and remove it for the summary SKIP=cargo-fmt,clippy,dev-generate-all pre-commit run --all-files --show-diff-on-failure --color=always | \ tee >(sed -E 's/\x1B\[([0-9]{1,2}(;[0-9]{1,2})*)?[mGK]//g' >> $GITHUB_STEP_SUMMARY) >&1 + exit_code=${PIPESTATUS[0]} echo '```' >> $GITHUB_STEP_SUMMARY + exit $exit_code diff --git a/scripts/pyproject.toml b/scripts/pyproject.toml index ee780c6e0e..a7ac49add4 100644 --- a/scripts/pyproject.toml +++ b/scripts/pyproject.toml @@ -11,8 +11,9 @@ line-length = 88 line-length = 88 select = ["ALL"] ignore = [ - "PL", # pylint - "S", # bandit + "C901", # McCabe complexity + "PL", # pylint + "S", # bandit ] [tool.ruff.pydocstyle]