Skip to content

Commit d91d9be

Browse files
committed
.ci/check-tidy: fix early abort and don't lint tests
The check looped both build and build-build, but aborted if for build, there were no sources to lint. This way, in many PRs the linter missed linting the tests, which now have many warnings/errors. We disable these for now to have CI pass and can fix the individual issues at a later time.
1 parent a79d4b8 commit d91d9be

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

.ci/check-tidy

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,7 @@ for dir in build build-build; do
3838
# Only check our files
3939
SOURCES1=$(git --no-pager diff --diff-filter=d --name-only ${TARGET_BRANCH} |\
4040
grep -v -E "(^src/(drivers|ui/fonts)|.*ugui.*|.*base32.*)" |\
41-
grep -E "^(src|test)" |\
41+
grep -E "^(src)" |\
4242
grep -v "^test/unit-test/u2f/" |\
4343
grep -E "\.c\$" || true)
4444

@@ -51,7 +51,7 @@ for dir in build build-build; do
5151
done
5252

5353
if [ -z "${SOURCES}" ] ; then
54-
exit 0
54+
continue
5555
fi
5656

5757
echo "Checking $(echo ${SOURCES} | wc -w) files with clang-tidy"

0 commit comments

Comments
 (0)