-
Couldn't load subscription status.
- Fork 411
Merge main into releases/v4 #3235
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
- Throws a `ConfigurationError` if parsing the YAML fails - Add a couple of tests for it
Mergeback v4.30.9 refs/heads/releases/v4 into main
Co-authored-by: Copilot <[email protected]>
Co-authored-by: Michael B. Gale <[email protected]>
Add experimental functionality for labelling PRs by their size
Bumps the npm-minor group with 5 updates: | Package | From | To | | --- | --- | --- | | [octokit](https://github.com/octokit/octokit.js) | `5.0.3` | `5.0.4` | | [@eslint/js](https://github.com/eslint/eslint/tree/HEAD/packages/js) | `9.37.0` | `9.38.0` | | [@typescript-eslint/eslint-plugin](https://github.com/typescript-eslint/typescript-eslint/tree/HEAD/packages/eslint-plugin) | `8.46.0` | `8.46.1` | | [@typescript-eslint/parser](https://github.com/typescript-eslint/typescript-eslint/tree/HEAD/packages/parser) | `8.46.0` | `8.46.1` | | [esbuild](https://github.com/evanw/esbuild) | `0.25.10` | `0.25.11` | Updates `octokit` from 5.0.3 to 5.0.4 - [Release notes](https://github.com/octokit/octokit.js/releases) - [Commits](octokit/octokit.js@v5.0.3...v5.0.4) Updates `@eslint/js` from 9.37.0 to 9.38.0 - [Release notes](https://github.com/eslint/eslint/releases) - [Commits](https://github.com/eslint/eslint/commits/v9.38.0/packages/js) Updates `@typescript-eslint/eslint-plugin` from 8.46.0 to 8.46.1 - [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.46.1/packages/eslint-plugin) Updates `@typescript-eslint/parser` from 8.46.0 to 8.46.1 - [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.46.1/packages/parser) Updates `esbuild` from 0.25.10 to 0.25.11 - [Release notes](https://github.com/evanw/esbuild/releases) - [Changelog](https://github.com/evanw/esbuild/blob/main/CHANGELOG.md) - [Commits](evanw/esbuild@v0.25.10...v0.25.11) --- updated-dependencies: - dependency-name: octokit dependency-version: 5.0.4 dependency-type: direct:production update-type: version-update:semver-patch dependency-group: npm-minor - dependency-name: "@eslint/js" dependency-version: 9.38.0 dependency-type: direct:development update-type: version-update:semver-minor dependency-group: npm-minor - dependency-name: "@typescript-eslint/eslint-plugin" dependency-version: 8.46.1 dependency-type: direct:development update-type: version-update:semver-patch dependency-group: npm-minor - dependency-name: "@typescript-eslint/parser" dependency-version: 8.46.1 dependency-type: direct:development update-type: version-update:semver-patch dependency-group: npm-minor - dependency-name: esbuild dependency-version: 0.25.11 dependency-type: direct:development update-type: version-update:semver-patch dependency-group: npm-minor ... Signed-off-by: dependabot[bot] <[email protected]>
Bumps [actions/setup-node](https://github.com/actions/setup-node) from 5 to 6. - [Release notes](https://github.com/actions/setup-node/releases) - [Commits](actions/setup-node@v5...v6) --- updated-dependencies: - dependency-name: actions/setup-node dependency-version: '6' dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <[email protected]>
…thub/workflows/actions/setup-node-6 Bump actions/setup-node from 5 to 6 in /.github/workflows
…r-5ed6ededba Bump the npm-minor group with 5 updates
Handle user errors for invalid `UserConfig`s and missing query files
Bump timeout for `analyze-action-env` test
Update wording in some log messages
…-limit Overlay: Lower size limit for overlay base databases
Perform SARIF post-processing independently of upload
Also add tests for it and `getRequiredEnvParam`
Bump minimum CodeQL Bundle version to 2.17.6
Add `getOptionalEnvVar` helper
Add changelog entry for post-processing change
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This is a release PR merging changes from main into releases/v4 for version 4.31.0. The PR includes multiple merged pull requests that implement improvements to SARIF file processing, error handling, dependency updates, and various code quality enhancements.
Key changes:
- Bumped minimum CodeQL bundle version from 2.16.6 to 2.17.6
- Refactored SARIF upload logic to always perform post-processing, even when uploads are disabled
- Added new
post-processed-sarif-pathinput to the analyze action for saving post-processed SARIF files
Reviewed Changes
Copilot reviewed 63 out of 66 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| package.json | Version bump to 4.31.0 and dependency updates |
| CHANGELOG.md | Release notes for version 4.31.0 |
| src/upload-sarif.ts | Renamed function and added post-processing support with upload kind parameter |
| src/upload-lib.ts | Refactored to separate post-processing from upload logic |
| src/analyze-action.ts | Updated to use new upload architecture with conditional upload |
| src/util.ts | Added asHTTPError helper and getOptionalEnvVar function |
| src/codeql.ts | Bumped minimum CodeQL version constant |
| src/workflow.ts, src/tar.ts, etc. | Applied optional chaining improvements |
| analyze/action.yml | Added new post-processed-sarif-path input |
| .github/workflows/*.yml | Updated setup-node action from v5 to v6 |
| lib/*.js | Generated JavaScript from TypeScript sources |
Merging 1d36546 into
releases/v4.Conductor for this PR is @mbg.
Contains the following pull requests:
UserConfigs and missing query files #3203 (@mbg)uploadSarifrather thanuploadFilesinanalyzeaction #3206 (@mbg)AnalysisKind.CodeScanningis not enabled #3221 (@mbg)analyze-action-envtest #3228 (@mbg)getOptionalEnvVarhelper #3233 (@mbg)Please do the following:
releases/v4branch.Create a merge commitis selected rather thanSquash and mergeorRebase and merge.