Skip to content

ci: CodeQL advanced setup so fork pull requests can satisfy the gate - #58

Merged
MohammadHaroonAbuomar merged 4 commits into
mainfrom
codeql-advanced-setup
Jul 30, 2026
Merged

ci: CodeQL advanced setup so fork pull requests can satisfy the gate#58
MohammadHaroonAbuomar merged 4 commits into
mainfrom
codeql-advanced-setup

Conversation

@MohammadHaroonAbuomar

Copy link
Copy Markdown
Contributor

CodeQL default setup never creates its check runs on fork pull requests, so the six required Analyze (*) contexts are unsatisfiable for any external contribution (live example: #57, approved and otherwise green, refused with "6 of 17 required status checks are expected"). This adds the advanced-setup workflow — six languages, build-mode none, pull_request + main + weekly triggers — whose CodeQL (<language>) contexts report on every head, fork or branch. Follow-up (repo settings, after this merges and proves green on main): swap the required contexts and disable default setup to avoid double analysis.

Default-setup check runs are never created on fork pull requests, so
the six required Analyze contexts are unsatisfiable for external
contributors. The advanced-setup workflow runs on every pull request
and push to main; its CodeQL (<language>) contexts replace the
default-setup ones in branch protection.

Signed-off-by: MohammadHaroonAbuomar <40180927+MohammadHaroonAbuomar@users.noreply.github.com>
Signed-off-by: MohammadHaroonAbuomar <40180927+MohammadHaroonAbuomar@users.noreply.github.com>
Signed-off-by: MohammadHaroonAbuomar <40180927+MohammadHaroonAbuomar@users.noreply.github.com>
The Go extractor traces the toolchain present at init time; installing
Go afterwards leaves the tracer with nothing to wrap (no source seen).

Signed-off-by: MohammadHaroonAbuomar <40180927+MohammadHaroonAbuomar@users.noreply.github.com>
@MohammadHaroonAbuomar
MohammadHaroonAbuomar merged commit 4801a48 into main Jul 30, 2026
29 checks passed
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.

1 participant