Skip to content

Use the existing CI container for static analysis - #57

Merged
mattia-moffa merged 2 commits into
wolfSSL:mainfrom
aidangarske:fix/coverity-ci-container
Oct 1, 2026
Merged

mattia-moffa merged 2 commits into
wolfSSL:mainfrom
aidangarske:fix/coverity-ci-container

Conversation

@aidangarske

Copy link
Copy Markdown
Member
  • use the ghcr container for coverity

Copilot AI balanced review requested due to automatic review settings October 1, 2026 17:41

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.

Copilot review overview

🟢 Approval recommended

The workflows consistently reuse the established image while retaining required host and ARM toolchain validation.

Review effort: Balanced
Findings: None

What changed in this PR

Uses the existing ARM CI image for CodeQL and Coverity analysis.

Changes:

  • Runs both workflows in the pinned M33MU container.
  • Replaces package installation with toolchain checks and dependency timeout safeguards.
File Description
.github/​workflows/​coverity.yml Runs Coverity in the CI container.
.github/​workflows/​codeql.yml Runs CodeQL in the CI container.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@mattia-moffa
mattia-moffa merged commit 5d36708 into wolfSSL:main Oct 1, 2026
104 checks passed
@aidangarske
aidangarske deleted the fix/coverity-ci-container branch October 1, 2026 18:27
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