Merged
Conversation
This triggers a subset of the behavior of `CODEQL_ACTION_TEST_MODE`, specifically just skipping the SARIF upload step. This is required for our internal testing where we want the SARIF file (via `CODEQL_ACTION_DUMP_SARIF_DIR`) but don't want to actually upload it, but we don't want the rest of the behaviour of `CODEQL_ACTION_TEST_MODE` that is specific for `codeql-action` own CI checks.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
This reverts commit 680b070.
Bumps the npm group with 4 updates: [@eslint/js](https://github.com/eslint/eslint/tree/HEAD/packages/js), [@typescript-eslint/eslint-plugin](https://github.com/typescript-eslint/typescript-eslint/tree/HEAD/packages/eslint-plugin), [@typescript-eslint/parser](https://github.com/typescript-eslint/typescript-eslint/tree/HEAD/packages/parser) and [typescript](https://github.com/microsoft/TypeScript). Updates `@eslint/js` from 9.36.0 to 9.37.0 - [Release notes](https://github.com/eslint/eslint/releases) - [Commits](https://github.com/eslint/eslint/commits/v9.37.0/packages/js) Updates `@typescript-eslint/eslint-plugin` from 8.44.1 to 8.45.0 - [Release notes](https://github.com/typescript-eslint/typescript-eslint/releases) - [Changelog](https://github.com/typescript-eslint/typescript-eslint/blob/main/packages/eslint-plugin/CHANGELOG.md) - [Commits](https://github.com/typescript-eslint/typescript-eslint/commits/v8.45.0/packages/eslint-plugin) Updates `@typescript-eslint/parser` from 8.44.1 to 8.45.0 - [Release notes](https://github.com/typescript-eslint/typescript-eslint/releases) - [Changelog](https://github.com/typescript-eslint/typescript-eslint/blob/main/packages/parser/CHANGELOG.md) - [Commits](https://github.com/typescript-eslint/typescript-eslint/commits/v8.45.0/packages/parser) Updates `typescript` from 5.9.2 to 5.9.3 - [Release notes](https://github.com/microsoft/TypeScript/releases) - [Changelog](https://github.com/microsoft/TypeScript/blob/main/azure-pipelines.release-publish.yml) - [Commits](microsoft/TypeScript@v5.9.2...v5.9.3) --- updated-dependencies: - dependency-name: "@eslint/js" dependency-version: 9.37.0 dependency-type: direct:development update-type: version-update:semver-minor dependency-group: npm - dependency-name: "@typescript-eslint/eslint-plugin" dependency-version: 8.45.0 dependency-type: direct:development update-type: version-update:semver-minor dependency-group: npm - dependency-name: "@typescript-eslint/parser" dependency-version: 8.45.0 dependency-type: direct:development update-type: version-update:semver-minor dependency-group: npm - dependency-name: typescript dependency-version: 5.9.3 dependency-type: direct:development update-type: version-update:semver-patch dependency-group: npm ... Signed-off-by: dependabot[bot] <support@github.com>
…6854f6 Bump the npm group with 4 updates
Add more end-to-end tests for `upload-sarif`
Gate `tools: toolcache` behind FF
…thub/workflows/github/codeql-action-4 Bump github/codeql-action from 3 to 4 in /.github/workflows
…thub/workflows/actions-minor-945aab589d Bump ruby/setup-ruby from 1.263.0 to 1.265.0 in /.github/workflows in the actions-minor group across 1 directory
…r-37415c9066 Bump the npm-minor group with 3 updates
Merge main into releases/v4
Revert "Rebuild" commit rather than "Update dependencies"
This reverts commit b264e15.
This reverts commit c551c50.
henrymercer
added
Rebuild
Contributor
Author
|
Pushed a commit to rebuild the Action. Please mark the PR as ready for review to trigger PR checks. |
Contributor
There was a problem hiding this comment.
Pull Request Overview
This PR merges changes from the releases/v4 branch into releases/v3 to prepare version 3.30.8. The primary purpose is to backport improvements and fixes from v4 while maintaining compatibility with the v3 release line.
Key changes include:
- Added support for
tools: toolcacheinput to use the latest CodeQL CLI from the toolcache in dynamic workflows - Improved SARIF upload handling with more granular environment variable controls
- Enhanced error handling for rate-limited requests during CLI downloads
- Updated test infrastructure and dependency versions
Reviewed Changes
Copilot reviewed 47 out of 48 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| src/util.ts | Added shouldSkipSarifUpload() function for better SARIF upload control |
| src/upload-sarif-action.ts | Updated to use new SARIF upload control function |
| src/upload-lib.ts | Enhanced payload upload function with better error handling and analysis-specific naming |
| src/setup-codeql.ts | Added toolcache input support and latest version detection functionality |
| src/start-proxy-action.ts | Improved language parameter handling and status reporting |
| src/codeql.ts | Enhanced error handling for RequestError with 429 status codes |
| package.json | Updated to version 3.30.8 and various dependency updates |
| CHANGELOG.md | Added entry for version 3.30.8 |
mbg
approved these changes
Oct 10, 2025
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Merging 2a6736c into
releases/v3.Conductor for this PR is @henrymercer.
Contains the following pull requests:
tools: toolcache#3175 (@mbg)CODEQL_ACTION_SKIP_SARIF_UPLOAD#3180 (@redsun82)upload-sarif#3181 (@mbg)uploadPayload#3185 (@redsun82)Partial<Config>forcreateStatusReportBase#3188 (@mbg)tools: toolcachebehind FF #3193 (@mbg)Please do the following:
releases/v3branch.Create a merge commitis selected rather thanSquash and mergeorRebase and merge.