Skip to content

Commit 5cedce0

Browse files
committed
Try other approach
1 parent f5ccbe6 commit 5cedce0

File tree

1 file changed

+18
-7
lines changed

1 file changed

+18
-7
lines changed

.github/workflows/lint.yaml

Lines changed: 18 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -24,15 +24,26 @@ jobs:
2424
2525
- name: Analyzing the code with Buildifier
2626
run: |
27+
set +e
28+
BUILDIFIER_FORMAT=$(buildifier -diff_command="diff -u" -d -r . 2>&1)
29+
FORMAT_EXIT_CODE=$?
30+
BUILDIFIER_LINT=$(buildifier --lint=warn -r . 2>&1)
31+
LINT_EXIT_CODE=$?
32+
set -e
33+
echo "$BUILDIFIER_FORMAT"
34+
echo "$BUILDIFIER_LINT"
2735
echo "### Buildifier Format Issues" >> $GITHUB_STEP_SUMMARY
28-
echo '```diff' >> $GITHUB_STEP_SUMMARY
29-
buildifier -diff_command="diff -u" -d -r . 2>&1 | tee -a $GITHUB_STEP_SUMMARY
30-
echo '```' >> $GITHUB_STEP_SUMMARY
36+
echo "```diff" >> $GITHUB_STEP_SUMMARY
37+
echo "$BUILDIFIER_FORMAT" >> $GITHUB_STEP_SUMMARY
38+
echo "```" >> $GITHUB_STEP_SUMMARY
3139
echo "### Buildifier Linter Issues" >> $GITHUB_STEP_SUMMARY
3240
echo '```diff' >> $GITHUB_STEP_SUMMARY
33-
buildifier --lint=warn -r . 2>&1 | tee -a $GITHUB_STEP_SUMMARY
34-
echo '```' >> $GITHUB_STEP_SUMMARY
35-
# To fail job if either format or lint errors were found
36-
buildifier --mode=diff --lint=warn -r .
41+
echo "$BUILDIFIER_LINT" >> $GITHUB_STEP_SUMMARY
42+
eho '```' >> $GITHUB_STEP_SUMMARY
43+
if [ $FORMAT_EXIT_CODE -ne 0 ]; then
44+
exit $FORMAT_EXIT_CODE
45+
else
46+
exit $LINT_EXIT_CODE
47+
fi
3748
3849
# TODO: Add more linters here

0 commit comments

Comments
 (0)