Skip to content

Commit

Permalink
Merge bitcoin#22381: guix: Test security-check sanity before performi…
Browse files Browse the repository at this point in the history
…ng them

BACKPORT NOTICE: it includes missing commit: d6ef354

lint: Run mypy with --show-error-codes

When using mypy ignore directives, the error code needs to be specified.
Somehow mypy doesn't print it by default...
  • Loading branch information
dongcarl authored and knst committed Sep 25, 2024
1 parent 9ef68d1 commit b609514
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion test/lint/lint-python.sh
Original file line number Diff line number Diff line change
Expand Up @@ -111,7 +111,7 @@ if ! PYTHONWARNINGS="ignore" $FLAKECMD --ignore=B,C,E,F,I,N,W --select=$(IFS=","
EXIT_CODE=1
fi

if ! mypy --ignore-missing-imports $(git ls-files "test/functional/*.py" "contrib/devtools/*.py" | grep -v contrib/devtools/github-merge.py) ; then
if ! mypy --ignore-missing-imports --show-error-codes $(git ls-files "test/functional/*.py" "contrib/devtools/*.py" | grep -v contrib/devtools/github-merge.py) ; then
EXIT_CODE=1
fi

Expand Down

0 comments on commit b609514

Please sign in to comment.