Skip to content

Merge main into releases/v3 #2926

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 22 commits into from
Jun 11, 2025
Merged

Merge main into releases/v3 #2926

merged 22 commits into from
Jun 11, 2025

Conversation

github-actions[bot]
Copy link
Contributor

@github-actions github-actions bot commented Jun 11, 2025 •
edited by cklin
Loading

Merging e879928 into releases/v3.

Conductor for this PR is @cklin.

Contains the following pull requests:

Please do the following:

  • Ensure the CHANGELOG displays the correct version and date.
  • Ensure the CHANGELOG includes all relevant, user-facing changes since the last release.
  • Check that there are not any unexpected commits being merged into the releases/v3 branch.
  • Ensure the docs team is aware of any documentation changes that need to be released.
  • Mark the PR as ready for review to trigger the full set of PR checks.
  • Approve and merge this PR. Make sure Create a merge commit is selected rather than Squash and merge or Rebase and merge.
  • Merge the mergeback PR that will automatically be created once this PR is merged.
  • Merge all backport PRs to older release branches, that will automatically be created once this PR is merged.

henrymercer and others added 22 commits May 30, 2025 17:05
@henrymercer
Bump minimum CodeQL version to 2.16.6
fb70a8a
@henrymercer
Remove redundant code based on CLI version number
0be24c0
@henrymercer
Remove guards for fully enabled tool features
c7f3c79
@henrymercer
Update CodeQL versions in generated workflows
7419bc6
@henrymercer
Bump minor version number
36121ec
@henrymercer
Add changelog note
d54c5e2
@dependabot
build(deps): bump the npm group with 5 updates
55ff016 
Bumps the npm group with 5 updates:

| Package | From | To |
| --- | --- | --- |
| [octokit](https://github.com/octokit/octokit.js) | `5.0.2` | `5.0.3` |
| [@eslint/js](https://github.com/eslint/eslint/tree/HEAD/packages/js) | `9.27.0` | `9.28.0` |
| [@typescript-eslint/eslint-plugin](https://github.com/typescript-eslint/typescript-eslint/tree/HEAD/packages/eslint-plugin) | `8.32.1` | `8.33.1` |
| [@typescript-eslint/parser](https://github.com/typescript-eslint/typescript-eslint/tree/HEAD/packages/parser) | `8.32.1` | `8.33.1` |
| [nock](https://github.com/nock/nock) | `14.0.4` | `14.0.5` |


Updates `octokit` from 5.0.2 to 5.0.3
- [Release notes](https://github.com/octokit/octokit.js/releases)
- [Commits](octokit/octokit.js@v5.0.2...v5.0.3)

Updates `@eslint/js` from 9.27.0 to 9.28.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.28.0/packages/js)

Updates `@typescript-eslint/eslint-plugin` from 8.32.1 to 8.33.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.33.1/packages/eslint-plugin)

Updates `@typescript-eslint/parser` from 8.32.1 to 8.33.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.33.1/packages/parser)

Updates `nock` from 14.0.4 to 14.0.5
- [Release notes](https://github.com/nock/nock/releases)
- [Changelog](https://github.com/nock/nock/blob/main/CHANGELOG.md)
- [Commits](nock/nock@v14.0.4...v14.0.5)

---
updated-dependencies:
- dependency-name: octokit
  dependency-version: 5.0.3
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: npm
- dependency-name: "@eslint/js"
  dependency-version: 9.28.0
  dependency-type: direct:development
  update-type: version-update:semver-minor
  dependency-group: npm
- dependency-name: "@typescript-eslint/eslint-plugin"
  dependency-version: 8.33.1
  dependency-type: direct:development
  update-type: version-update:semver-minor
  dependency-group: npm
- dependency-name: "@typescript-eslint/parser"
  dependency-version: 8.33.1
  dependency-type: direct:development
  update-type: version-update:semver-minor
  dependency-group: npm
- dependency-name: nock
  dependency-version: 14.0.5
  dependency-type: direct:development
  update-type: version-update:semver-patch
  dependency-group: npm
...

Signed-off-by: dependabot[bot] <support@github.com>
@github-actions
Update checked-in dependencies
313daef
@github-actions
Update changelog and version after v3.28.19
a28627a
@github-actions
Update checked-in dependencies
be60d9f
@aibaars
Merge pull request #2919 from github/mergeback/v3.28.19-to-main-fca7ace9
075e08a 
Mergeback v3.28.19 refs/heads/releases/v3 into main
@mbg
Fix backslashes being accepted by sanitizeArtifactName
1eab408
@mbg
Merge pull request #2922 from github/mbg/fix/sanitizeArtifactName
72be4b6 
Fix backslashes being accepted by `sanitizeArtifactName`
@mbg
Merge pull request #2916 from github/dependabot/npm_and_yarn/npm-5cdc…
b1e4dc3 
…cdc43f

build(deps): bump the npm group with 5 updates
@aibaars
Merge branch 'main' into henrymercer/bump-minimum-codeql-2.16.6
df2a830
@aibaars
Update package-lock.json
22deae8
@github-actions
Update default bundle to codeql-bundle-v2.22.0
f5d4e2a
@github-actions
Add changelog note
3855117
@mbg
Merge pull request #2912 from github/henrymercer/bump-minimum-codeql-…
7cb9b16 
…2.16.6

Bump minimum CodeQL version to 2.16.6
@cklin
Merge branch 'main' into update-bundle/codeql-bundle-v2.22.0
efd43b3
@cklin
Merge pull request #2925 from github/update-bundle/codeql-bundle-v2.22.0
e879928 
Update default bundle to 2.22.0
@github-actions
Update changelog for v3.29.0
bc251b7
@cklin cklin marked this pull request as ready for review June 11, 2025 18:32
@Copilot Copilot AI review requested due to automatic review settings June 11, 2025 18:32
@cklin cklin requested a review from a team as a code owner June 11, 2025 18:32
Copilot
Copilot AI reviewed Jun 11, 2025
View reviewed changes
Copy link
Contributor

@Copilot Copilot AI left a 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 PR merges recent main changes into the releases/v3 branch, updating default versions, streamlining tracing and initialization logic, and cleaning up related tests.

  • Bumped default CodeQL bundle and CLI versions (2.22.0) in defaults.json and updated the changelog.
  • Simplified tracer configuration by removing explicit CODEQL_RUNNER injection and deprecated tests.
  • Removed printPathFiltersWarning and migrated initConfig to use consolidated logging groups.

Reviewed Changes

Copilot reviewed 1618 out of 1618 changed files in this pull request and generated no comments.

Show a summary per file
File Description
lib/defaults.json Updated bundle/CLI versions and prior version fields.
CHANGELOG.md Added new 3.29.0 entry reflecting version bumps.
lib/tracer-config.js Removed runner‐injection logic and unused feature checks.
lib/tracer-config.test.js Deleted deprecated assertions; renamed test.
lib/init.js Removed printPathFiltersWarning, migrated to withGroupAsync.
lib/init-action.js Consolidated debug artifact variables and suppressed outdated warnings.
lib/debug-artifacts.js Tightened regex in sanitizeArtifactName to strip backslashes.
lib/codeql.js Bumped minimum CodeQL version and removed build-mode feature guards.
lib/autobuild.js Removed unsupported feature checks around build modes.
lib/analyze.js Dropped redundant feature checks in extraction logic.
Comments suppressed due to low confidence (3)

lib/tracer-config.test.js:62

  • The renamed test no longer verifies that the returned environment includes the expected runner path for each platform. Consider restoring explicit assertions to validate the full env object when start-tracing.json is present.
(0, ava_1.default)("getCombinedTracerConfig", async (t) => {

lib/tracer-config.js:49

  • [nitpick] The comment mentions checking CLI support for trace-command --build-mode, but the corresponding feature check was removed. Update or remove this comment to avoid confusion.
// If the CLI supports `trace-command` with a `--build-mode`, we'll use direct tracing instead of

lib/init.js:43

  • The languages import is no longer used after removing printPathFiltersWarning. Consider deleting this import to clean up unused code.
const languages_1 = require("./languages");

@cklin cklin merged commit ce28f5b into releases/v3 Jun 11, 2025
268 checks passed
@cklin cklin deleted the update-v3.29.0-e8799281c branch June 11, 2025 18:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

Copilot code review Copilot Copilot left review comments

@cklin cklin cklin approved these changes

Assignees

@cklin cklin

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@cklin @henrymercer @aibaars @mbg