This is an automated email from the ASF dual-hosted git repository.
sjanc pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/mynewt-nimble.git
The following commit(s) were added to refs/heads/master by this push:
new e00b85ccf ci: Report style issues even with skip-style-check label
e00b85ccf is described below
commit e00b85ccfc49abbaf1731bb3eddaa6340d07ab52
Author: Szymon Janc <[email protected]>
AuthorDate: Thu Oct 22 14:23:12 2026 +0200
ci: Report style issues even with skip-style-check label
The skip-style-check label used to skip the formatting step entirely,
so style violations were not visible in the logs. Always run the check
and print the clang-format diff, but when the label is present only
emit a warning and do not fail the job.
Assisted-by: Claude:claude-opus-5-5
---
.github/workflows/compliance_check.yml | 12 +++++++-----
1 file changed, 7 insertions(+), 5 deletions(-)
diff --git a/.github/workflows/compliance_check.yml
b/.github/workflows/compliance_check.yml
index 17a4dcafd..ffcab13b3 100644
--- a/.github/workflows/compliance_check.yml
+++ b/.github/workflows/compliance_check.yml
@@ -64,8 +64,9 @@ jobs:
core.setOutput('skip', hasLabel);
- name: check style
- if: steps.label_check.outputs.skip != 'true'
shell: bash
+ env:
+ SKIP_STYLE_CHECK: ${{ steps.label_check.outputs.skip }}
run: |
set +e
INFO_URL="https://github.com/apache/mynewt-core/blob/master/CODING_STANDARDS.md";
@@ -78,15 +79,16 @@ jobs:
echo ""
echo "For formatting guidelines, see:"
echo " $INFO_URL"
+ if [ "$SKIP_STYLE_CHECK" = "true" ]; then
+ echo "::warning::Ignoring formatting issues because" \
+ "'skip-style-check' label is present."
+ exit 0
+ fi
exit 1
else
echo "All good, no formatting issues."
fi
- - name: Skip style-check if label present
- if: steps.label_check.outputs.skip == 'true'
- run: echo "Skipping style check because 'skip-style-check' label is
present."
-
style_license:
name: Licensing
runs-on: ubuntu-slim
