Update Coverity to run on main and release branches#3752
Update Coverity to run on main and release branches#3752
Conversation
There was a problem hiding this comment.
Pull request overview
This pull request restores a previously reverted change that updates the Coverity static analysis workflow to run on both the main branch and release branches. The original change (PR #3725) was reverted due to an error that was later determined to be unrelated to the change, as the error persisted even after the revert.
Changes:
- Updated the
COVERITY_SCAN_BRANCH_PATTERNenvironment variable to match both "main" and branches starting with "releases/" - This allows Coverity scans to run on release branches in addition to the main branch
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
@codex review |
|
Codex Review: Didn't find any major issues. Nice work! ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
dbabokin
left a comment
There was a problem hiding this comment.
LGTM
And again - would be good to fix zizmor error.
Description
The change was reverted due to the following error:
However, the failure continued to occur even after the revert, which indicates that the issue was not caused by this PR.
Timeline
Since the error persisted after the revert, the original change does not appear to be the root cause and should be restored.
Related Issue
Checklist
clang-format(e.g.,clang-format -i src/ispc.cpp)