Skip to content

ci: skip C++ build during CodeQL Actions analysis - #216

Merged
sjinks merged 1 commit into
masterfrom
ci/codeql-skip-cpp-build-for-actions
Nov 8, 2026
Merged

sjinks merged 1 commit into
masterfrom
ci/codeql-skip-cpp-build-for-actions

Conversation

@sjinks

@sjinks sjinks commented Nov 8, 2026

Copy link
Copy Markdown
Owner

Run dependency installation and the C++ build only for the c-cpp CodeQL matrix entry. The actions entry analyzes workflows without installing native dependencies or building the project; both CodeQL analyses remain enabled.

Install native dependencies and build only for C++ analysis; the
Actions analysis does not need either step.
Copilot AI balanced review requested due to automatic review settings November 8, 2026 01:42
@sonarqubecloud

sonarqubecloud Bot commented Nov 8, 2026

Copy link
Copy Markdown

@codecov

codecov Bot commented Nov 8, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 85.00%. Comparing base (82285b3) to head (65a7443).
⚠️ Report is 2 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #216   +/-   ##
=======================================
  Coverage   85.00%   85.00%           
=======================================
  Files           5        5           
  Lines         240      240           
  Branches       53       53           
=======================================
  Hits          204      204           
  Misses          1        1           
  Partials       35       35           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Approval recommended

No active review issues were identified for the CodeQL workflow change.

No open issues 🎉

What changed in this PR

This PR avoids unnecessary native build work during CodeQL workflow analysis while preserving both C++ and GitHub Actions analyses.

Changes:

  • Restricts dependency installation and the C++ build to the c-cpp matrix entry.
  • Leaves the actions entry enabled without running either step.
File Description
.github/​workflows/​codeql.yml Adds language conditions to the dependency installation and build steps.

Review effort: Balanced

@github-actions

github-actions Bot commented Nov 8, 2026

Copy link
Copy Markdown

Test Results

  5 files    5 suites   52s ⏱️
 71 tests  71 ✅ 0 💤 0 ❌
355 runs  355 ✅ 0 💤 0 ❌

Results for commit 65a7443.

@sjinks
sjinks merged commit 74c249c into master Nov 8, 2026
33 checks passed
@sjinks
sjinks deleted the ci/codeql-skip-cpp-build-for-actions branch November 8, 2026 02:08
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.

2 participants