Skip to content

chore(ci): atomically pin CodeQL init/autobuild/analyze to v4.39.0 - #342

Merged
akhiabanchian merged 1 commit into
mainfrom
fix/ci-codeql-4390-atomic
Oct 21, 2026
Merged

akhiabanchian merged 1 commit into
mainfrom
fix/ci-codeql-4390-atomic

Conversation

@ammarheidari

Copy link
Copy Markdown
Contributor

Resolve PRs #333 #334 #335 together

Each individual Dependabot PR upgrades only one github/codeql-action step. GitHub CodeQL explicitly fails with configuration error when init, autobuild, and analyze resolve to different versions (observed in #333/#334/#335 exact-head CodeQL logs). This PR changes all three to the same verified commit 6f2ac81f5cd81fe351a197ba0b00d91d03cd2aee (v4.39.0) in .github/workflows/codeql.yml.

No CodeQL suppression, skipped checks or product/version scope change. Do not merge unless exact-head C#/JS CodeQL, quality/benchmark/dependency checks, CODEOWNER and Codex review all pass. Once this atomic replacement is admitted, close individual #333–#335 as superseded, not merged.

Signed-off-by: Ammar Heidari ammar@arad-itc.org

Prevent mixed action version configuration errors in init/autobuild/analyze.

Signed-off-by: Ammar Heidari <ammar@arad-itc.org>
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 19, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-19T06:51:21.723335Z 8f4ead9 PR opened
ℹ️ 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" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

Copy link
Copy Markdown
Contributor Author

Exact-head final CI verification on 8f4ead906375cbb8ed4461062b43280fb1513686 (2026-10-19): all reported checks SUCCESS, including CodeQL C# and JavaScript, aggregate CodeQL, quality-gate, dependency-review, benchmarks, real Keycloak/Webhook, human CLI/transports, PostgreSQL fencing/restore, HA replay and Windows/macOS/Linux loopback. Codex review COMPLETE without findings; no unresolved review threads. Only fresh CODEOWNER approval remains, so this PR must NOT be merged yet; expected-head protected-main admission follows that approval. Individual Dependabot PRs #333–#335 were already closed unmerged as mutually incompatible standalone updates.

@akhiabanchian
akhiabanchian merged commit b38da46 into main Oct 21, 2026
15 checks passed
@akhiabanchian
akhiabanchian deleted the fix/ci-codeql-4390-atomic branch October 21, 2026 07:13
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