| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
Although the workflow diff is straightforward, safe merging requires removing both macos-14 matrix contexts from external branch-protection required checks, which cannot be verified from the checked-in files.
No open issues 🎉
What changed in this PRRemoves the unsupported macos-14-xlarge runner from the CodeQL JavaScript analysis matrix, preventing required merge-queue checks from failing.
Changes:
| File | Description |
|---|---|
| .github/workflows/codeql.yml | Removes the unsupported macOS 14 runner from the CodeQL analysis matrix. |
Review effort: Lite (auto)
Sorry, something went wrong.
|
Should we add macos-26? |
Sorry, something went wrong.
I plan to rework the checks so we make sure each runner image is tested, but not necessarily on the JS analysis for this repo. I'll do that in a separate PR though so we can unblock CI faster. (Note for instance that Swift analysis on macos-26 is already covered by this check: https://github.com/github/codeql-action/actions/runs/42269788821/job/128071063024?pr=4309) |
Sorry, something went wrong.
| Back | FazBrowse Home | New Git URL |
The analyze-javascript (macos-14-xlarge) jobs in the codeql workflow are now cancelled with "The configuration 'macos-14-arm64-us-xl' is not supported". These are required checks, so every PR in the merge queue is failing. This PR removes macos-14-xlarge without a replacement, since many other PR checks already run on macOS 26. A follow-up PR will look at the overall PR check coverage across runner images.
Risk assessment
For internal use only. Please select the risk level of this change:
Which use cases does this change impact?
Environments:
How did/will you validate this change?
If something goes wrong after this change is released, what are the mitigation and rollback strategies?
How will you know if something goes wrong after this change is released?
Are there any special considerations for merging or releasing this change?
Merge / deployment checklist