Skip to content

fix(ci): remove workflow CodeQL — default setup is enabled - #181

Merged
ronaldtse merged 1 commit into
mainfrom
fix/codeql-default-setup
Aug 25, 2026
Merged

ronaldtse merged 1 commit into
mainfrom
fix/codeql-default-setup

Conversation

@ronaldtse

Copy link
Copy Markdown
Contributor

CodeQL default setup is enabled on this repo, and GitHub rejects SARIF uploads from workflow-based (advanced) configurations with it enabled. The default-setup workflow already runs on every push, so the workflow copy is redundant and permanently red. Removing it.

GitHub rejects SARIF from advanced (workflow) configurations when
default setup is enabled; default setup already scans every push.
@ronaldtse

Copy link
Copy Markdown
Contributor Author

The failing test (3.3/3.4) jobs on this PR are unrelated to its one-file diff: the job clones the monorepo and runs its ruby suite, and the monorepo's js/ removal (44cef0a, "to be added as a submodule in a moment") broke the interpreter specs (js/test-compiler/xregexp.js no longer exists on monorepo main). PR #180 retires that proxy job from this repo's CI in favor of parse-all jobs over the .isc corpus, so this should unblock once #180 lands.

@ronaldtse
ronaldtse merged commit b076db5 into main Aug 25, 2026
3 of 7 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