Skip to content

Update Coverity to run on main and release branches#3752

Open
azwolski wants to merge 1 commit intomainfrom
restore-coverity
Open

Update Coverity to run on main and release branches#3752
azwolski wants to merge 1 commit intomainfrom
restore-coverity

Conversation

@azwolski
Copy link
Collaborator

@azwolski azwolski commented Feb 24, 2026

Description

The change was reverted due to the following error:

parse error: Invalid numeric literal at line 2, column 0
Error: Process completed with exit code 4.

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

  • Code has been formatted with clang-format (e.g., clang-format -i src/ispc.cpp)
  • Git history has been squashed to meaningful commits (one commit per logical change)
  • Compiler changes are covered by lit tests
  • Language/stdlib changes include new functional tests for runtime behavior
  • Documentation updated if needed

@azwolski azwolski marked this pull request as ready for review February 25, 2026 14:57
@azwolski azwolski requested a review from Copilot February 27, 2026 12:02
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

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_PATTERN environment 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.

@dbabokin
Copy link
Collaborator

@codex review

@chatgpt-codex-connector
Copy link

Codex Review: Didn't find any major issues. Nice work!

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

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".

Copy link
Collaborator

@dbabokin dbabokin left a comment

Choose a reason for hiding this comment

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

LGTM

And again - would be good to fix zizmor error.

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