Skip to content

Add clang-tidy run to nightly workflow#498

Merged
limdor merged 11 commits into
eclipse-score:mainfrom
ahmed0mousa:ahmo_add_clang_to_nightly_workflow
Jun 3, 2026
Merged

Add clang-tidy run to nightly workflow#498
limdor merged 11 commits into
eclipse-score:mainfrom
ahmed0mousa:ahmo_add_clang_to_nightly_workflow

Conversation

@ahmed0mousa
Copy link
Copy Markdown
Contributor

No description provided.

…ly_quality

Set conclusion and output variables to share the result with the calling workflow
skip Fail check if clang-tidy errors found for nightly job as we will make it impossible to distinguish "the pipeline infrastructure failed" from "we have code quality issues to track.
add clang-tidy workflow to be invoked during the nightly run, download the result, and add it to the dashboard generation
…/<artifact-name>/,

but the directory name must have differed from nightly-quality-reports.
…f earlier failures

in this case no need for  continue-on-error: true in Run Unit Test step
head closes the pipe early, causing find to get SIGPIPE. prevents set -e from failing the step
use `outputs.conclusion` instead of `result` for all
three quality jobs — `result` is unreliable when `continue-on-error: true`
is in use; guard Extract step on `steps.download-coverage.outcome` to
prevent crashing when the download itself fails; download clang-tidy and
CodeQL findings to /tmp (not _quality/) since they are only needed by the
dashboard generator and must not be deployed to GitHub Pages; replace the
dead-code Resolve step with `id: setup` on Allow linux-sandbox, used as a
clean infrastructure checkpoint for the dashboard and upload steps
@ahmed0mousa ahmed0mousa force-pushed the ahmo_add_clang_to_nightly_workflow branch from af15216 to 8deb959 Compare June 3, 2026 09:55
@limdor limdor marked this pull request as ready for review June 3, 2026 10:25
@limdor limdor enabled auto-merge June 3, 2026 10:25
@limdor limdor added this pull request to the merge queue Jun 3, 2026
Merged via the queue into eclipse-score:main with commit 1a758d1 Jun 3, 2026
11 checks passed
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.

3 participants