Skip to content

Add static analysis for FreeRTOS#39

Merged
kubasejdak merged 15 commits into
mainfrom
maintenance/check-clang-rules
May 2, 2026
Merged

Add static analysis for FreeRTOS#39
kubasejdak merged 15 commits into
mainfrom
maintenance/check-clang-rules

Conversation

@kubasejdak
Copy link
Copy Markdown
Member

No description provided.

kubasejdak and others added 11 commits April 22, 2026 22:24
…ives

The previous `grep -qvE` command exits 0 when any line does NOT match the
failure pattern. If toJSON() emits multi-line JSON, bracket lines like `[`
and `]` pass the filter and cause the check to incorrectly report success
even when a dependency job failed.

Switch to `! grep -qE` which exits non-zero when any line contains a failure
indicator, regardless of JSON formatting.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@kubasejdak kubasejdak changed the title Remove NOLINT Add static analysis for May 2, 2026
@kubasejdak kubasejdak changed the title Add static analysis for Add static analysis for freertos May 2, 2026
@kubasejdak kubasejdak changed the title Add static analysis for freertos Add static analysis for FreeRTOS May 2, 2026
@kubasejdak kubasejdak enabled auto-merge (squash) May 2, 2026 22:42
@kubasejdak kubasejdak merged commit cac75f7 into main May 2, 2026
32 checks passed
@kubasejdak kubasejdak deleted the maintenance/check-clang-rules branch May 2, 2026 22:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant