Skip to content

Commit 0e3b1f5

Browse files
committed
ci: make yamllint report as success if no yaml files changed
Require to block PR merging
1 parent b21ae32 commit 0e3b1f5

File tree

2 files changed

+9
-3
lines changed

2 files changed

+9
-3
lines changed

.github/workflows/clang_tidy.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ jobs:
1313
- name: Run Clang-Tidy
1414
run: |
1515
CHANGED_FILES=$(git diff --name-only ${{ github.event.before }} ${{ github.sha }})
16-
if echo "$CHANGED_FILES" | grep -E '\.(cpp|h|hpp|cc|cxx|hxx)$' > /dev/null; then
16+
if echo "$CHANGED_FILES" | grep -qE '\.(cpp|h|hpp|cc|cxx|hxx)$'; then
1717
echo -e "Running clang-tidy"
1818
bazel build //... \
1919
--aspects @bazel_clang_tidy//clang_tidy:clang_tidy.bzl%clang_tidy_aspect \

.github/workflows/yamllint.yml

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22
name: Lint YAML files
33
on: # yamllint disable-line rule:truthy
44
pull_request:
5-
paths: ['**/*.yml', '**/*.yaml']
5+
branches: ['*']
66
jobs:
77
lint:
88
runs-on: ubuntu-latest
@@ -15,4 +15,10 @@ jobs:
1515
pip install yamllint
1616
- name: Lint YAML files
1717
run: |
18-
yamllint . || exit 1
18+
CHANGED_FILES=$(git diff --name-only ${{ github.event.before }} ${{ github.sha }})
19+
if echo "$CHANGED_FILES" | grep -qE '\.(yml|yaml)$'; then
20+
echo -e "Running yamllint"
21+
yamllint . || exit 1
22+
else
23+
echo -e "Skipping yamllint since no YAML files were changed."
24+
fi

0 commit comments

Comments
 (0)