Merged
Conversation
It isn't working, and this PR didn't break it. Let's fix it in a separate PR.
Mergeback v3.29.11 refs/heads/releases/v3 into main
This will make it easier to replace in the future
This doesn't give us much net new test coverage
Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
This means we avoid doing a type checking pass twice
Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
Bumps the actions group with 2 updates: [actions/setup-java](https://github.com/actions/setup-java) and [ruby/setup-ruby](https://github.com/ruby/setup-ruby). Updates `actions/setup-java` from 4 to 5 - [Release notes](https://github.com/actions/setup-java/releases) - [Commits](actions/setup-java@v4...v5) Updates `ruby/setup-ruby` from 1.255.0 to 1.256.0 - [Release notes](https://github.com/ruby/setup-ruby/releases) - [Changelog](https://github.com/ruby/setup-ruby/blob/master/release.rb) - [Commits](ruby/setup-ruby@829114f...efbf473) --- updated-dependencies: - dependency-name: actions/setup-java dependency-version: '5' dependency-type: direct:production update-type: version-update:semver-major dependency-group: actions - dependency-name: ruby/setup-ruby dependency-version: 1.256.0 dependency-type: direct:production update-type: version-update:semver-minor dependency-group: actions ... Signed-off-by: dependabot[bot] <support@github.com>
Bumps the npm group with 4 updates: [@types/node-forge](https://github.com/DefinitelyTyped/DefinitelyTyped/tree/HEAD/types/node-forge), [@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) and [@typescript-eslint/parser](https://github.com/typescript-eslint/typescript-eslint/tree/HEAD/packages/parser). Updates `@types/node-forge` from 1.3.13 to 1.3.14 - [Release notes](https://github.com/DefinitelyTyped/DefinitelyTyped/releases) - [Commits](https://github.com/DefinitelyTyped/DefinitelyTyped/commits/HEAD/types/node-forge) Updates `@eslint/js` from 9.33.0 to 9.34.0 - [Release notes](https://github.com/eslint/eslint/releases) - [Changelog](https://github.com/eslint/eslint/blob/main/CHANGELOG.md) - [Commits](https://github.com/eslint/eslint/commits/v9.34.0/packages/js) Updates `@typescript-eslint/eslint-plugin` from 8.40.0 to 8.41.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.41.0/packages/eslint-plugin) Updates `@typescript-eslint/parser` from 8.40.0 to 8.41.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.41.0/packages/parser) --- updated-dependencies: - dependency-name: "@types/node-forge" dependency-version: 1.3.14 dependency-type: direct:production update-type: version-update:semver-patch dependency-group: npm - dependency-name: "@eslint/js" dependency-version: 9.34.0 dependency-type: direct:development update-type: version-update:semver-minor dependency-group: npm - dependency-name: "@typescript-eslint/eslint-plugin" dependency-version: 8.41.0 dependency-type: direct:development update-type: version-update:semver-minor dependency-group: npm - dependency-name: "@typescript-eslint/parser" dependency-version: 8.41.0 dependency-type: direct:development update-type: version-update:semver-minor dependency-group: npm ... Signed-off-by: dependabot[bot] <support@github.com>
Pin Xcode 16
Add `analysis-kinds` input
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…c77296 Bump the npm group with 4 updates
…s-fd207b077b Bump the actions group with 2 updates
It's no longer needed since we aren't checking in `node_modules`.
Co-authored-by: Henry Mercer <henrymercer@github.com>
Bundle Actions using esbuild
Add workflow for rolling back release
Contributor
There was a problem hiding this comment.
Pull Request Overview
This is a release-PR that merges the main branch into releases/v3. The PR contains multiple JavaScript files being removed entirely from the lib directory, representing generated JavaScript code that is being cleaned up as part of the release process.
Key Changes
- Complete removal of multiple JavaScript files in the
libdirectory - Files removed include core action components like init, analyze, overlay database utilities, and various test files
- This appears to be a cleanup of compiled JavaScript code as part of the release preparation
Reviewed Changes
Copilot reviewed 78 out of 23867 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| lib/repository.js | Removed repository utility functions for GitHub repository handling |
| lib/overlay-database-utils.js | Removed overlay database management utilities |
| lib/overlay-database-utils.test.js | Removed corresponding test file |
| lib/logging.js | Removed logging functionality |
| lib/languages.js | Removed language definitions and enums |
| lib/init.js | Removed initialization action logic |
| lib/init.test.js | Removed initialization tests |
| lib/init-action-post-helper.js | Removed post-action helper utilities |
| lib/init-action-post-helper.test.js | Removed corresponding test file |
| lib/git-utils.js | Removed Git utility functions |
| lib/git-utils.test.js | Removed Git utility tests |
| lib/fingerprints.js | Removed SARIF fingerprinting logic |
| lib/fingerprints.test.js | Removed fingerprinting tests |
| lib/feature-flags.js | Removed feature flag management |
| lib/feature-flags.test.js | Removed feature flag tests |
| lib/environment.js | Removed environment variable definitions |
| lib/doc-url.js | Removed documentation URL constants |
| lib/diff-informed-analysis-utils.test.js | Removed diff-informed analysis tests |
henrymercer
approved these changes
Sep 1, 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
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 92eada8 into
releases/v3.Conductor for this PR is @henrymercer.
Contains the following pull requests:
analysis-kindsinput #3061 (@mbg)Please do the following:
releases/v3branch.Create a merge commitis selected rather thanSquash and mergeorRebase and merge.