From f5a17971bdea014dde8093d033918aaf34446523 Mon Sep 17 00:00:00 2001 From: Renzo F <45401804+rennf93@users.noreply.github.com> Date: Mon, 2 Feb 2026 03:30:01 +0100 Subject: [PATCH 1/2] Simplify pre-commit checks in scheduled lint workflow Removed unnecessary output handling and exit code checks from pre-commit run step. --- .github/workflows/scheduled-lint.yml | 26 ++------------------------ 1 file changed, 2 insertions(+), 24 deletions(-) diff --git a/.github/workflows/scheduled-lint.yml b/.github/workflows/scheduled-lint.yml index 680be84..79e5024 100644 --- a/.github/workflows/scheduled-lint.yml +++ b/.github/workflows/scheduled-lint.yml @@ -52,21 +52,7 @@ jobs: - name: Run pre-commit checks id: precommit - run: | - uv run pre-commit run --all-files > precommit_output.txt 2>&1 - EXIT_CODE=$? - - OUTPUT=$(cat precommit_output.txt) - echo "output<> $GITHUB_OUTPUT - echo "$OUTPUT" >> $GITHUB_OUTPUT - echo "EOF" >> $GITHUB_OUTPUT - - if [ $EXIT_CODE -eq 0 ]; then - echo "status=success" >> $GITHUB_OUTPUT - else - echo "status=failed" >> $GITHUB_OUTPUT - exit 1 - fi + run: uv run pre-commit run --all-files - name: Notify Success on Slack Channel uses: rennf93/good-comms@master @@ -83,10 +69,6 @@ jobs: MESSAGE: | . **All checks passed!** - - ``` - ${{ steps.precommit.outputs.output }} - ``` . COLOR: good SLACK_THREAD_TS: ${{ steps.send_initial_slack_lint.outputs.SLACK_THREAD_TS }} @@ -108,10 +90,6 @@ jobs: . **MYPY AND RUFF ARE OUTDATED!** - ``` - ${{ steps.precommit.outputs.output }} - ``` - Run `make fix` | `make lint` to try, check and resolve. . COLOR: danger @@ -220,4 +198,4 @@ jobs: Run `make fix-docs` | `make lint-docs` to resolve. . COLOR: danger - SLACK_THREAD_TS: ${{ steps.send_initial_slack_docs_lint.outputs.SLACK_THREAD_TS }} \ No newline at end of file + SLACK_THREAD_TS: ${{ steps.send_initial_slack_docs_lint.outputs.SLACK_THREAD_TS }} From 134229262e6487da2334e254d439adf978662a6f Mon Sep 17 00:00:00 2001 From: Renzo F <45401804+rennf93@users.noreply.github.com> Date: Mon, 2 Feb 2026 03:32:57 +0100 Subject: [PATCH 2/2] Disable semgrep in pre-commit configuration Comment out semgrep configuration in pre-commit. --- .pre-commit-config.yaml | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml index 23658c8..fefd223 100644 --- a/.pre-commit-config.yaml +++ b/.pre-commit-config.yaml @@ -60,10 +60,10 @@ repos: require_serial: true language: system pass_filenames: false - - id: semgrep - name: semgrep - entry: uv run semgrep --config=auto --error guard - require_serial: true - language: system - types: [python] - pass_filenames: false \ No newline at end of file + # - id: semgrep + # name: semgrep + # entry: uv run semgrep --config=auto --error guard + # require_serial: true + # language: system + # types: [python] + # pass_filenames: false