eng: update eng dependencies to latest versions#2364
Draft
bwateratmsft wants to merge 1 commit into
Draft
Conversation
Bump @vscode/test-cli (^0.0.12 -> ^0.0.15), tsx (^4.22.4 -> ^4.23.0), and eslint (^10.5.0 -> ^10.6.0) to their latest available versions. Leaves TypeScript and @types/node untouched. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Contributor
There was a problem hiding this comment.
Pull request overview
Updates eng/ package tooling dependencies to newer patch versions to keep the engineering/lint/test toolchain current.
Changes:
- Bumped
eslintfrom^10.5.0to^10.6.0. - Bumped peer deps
@vscode/test-clito^0.0.15andtsxto^4.23.0. - Updated
eng/package-lock.jsonto reflect the new dependency resolutions.
Reviewed changes
Copilot reviewed 1 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| eng/package.json | Updates tooling dependency and peerDependency versions. |
| eng/package-lock.json | Updates lockfile entries to match the new versions (includes a notable registry/integrity change for eslint). |
Files not reviewed (1)
- eng/package-lock.json: Generated file
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Comment on lines
1830
to
+1833
| "node_modules/eslint": { | ||
| "version": "10.5.0", | ||
| "resolved": "https://registry.npmjs.org/eslint/-/eslint-10.5.0.tgz", | ||
| "integrity": "sha512-1y+7C+vi12bUK1IpZeaV3gsH9fHLBmPvYmPx42pvT/E9yG0IC8g3PUZZgp0+JLJl7ZDK0flc2gc+Aw9dpCvIsQ==", | ||
| "version": "10.6.0", | ||
| "resolved": "https://ms-feed-25.pkgs.visualstudio.com/1es-public/_packaging/npm-public/npm/registry/eslint/-/eslint-10.6.0.tgz", | ||
| "integrity": "sha1-4bQFnFgr6VDHCIybVfmEc4skPCc=", |
bwateratmsft
changed the title
bwateratmsft
commented
Jul 13, 2026
| "eslint": "^10.6.0", | ||
| "mocha": "^11.7.6", | ||
| "typescript": "~5.9", | ||
| "typescript-eslint": "^8.61.1" |
Contributor
Author
There was a problem hiding this comment.
Should update this too.
bwateratmsft
commented
Jul 13, 2026
| "eslint": "^10.5.0", | ||
| "eslint": "^10.6.0", | ||
| "mocha": "^11.7.6", | ||
| "typescript": "~5.9", |
Contributor
Author
There was a problem hiding this comment.
Thinking about moving to v6 right now...dunno yet. Or maybe just straight to 7.
Contributor
Author
There was a problem hiding this comment.
Down side of v7 is that eslint still needs v6, until 7.1 is released adding the API.
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.
Updates dependencies in the
engpackage to their latest available versions.Changes
@vscode/test-cli(peer)^0.0.12^0.0.15tsx(peer)^4.22.4^4.23.0eslint^10.5.0^10.6.0Other test/build tooling (
@vscode/test-electron,@vscode/vsce,esbuild,esbuild-plugin-copy,mocha,chai) was already at its latest version.Intentionally left untouched:
7.0.1-rc); staying on~5.9.typescript-eslint— TypeScript-adjacent tooling, deferred.@types/node— tracks the Node major (latest is26.x); staying on22.x.Verification
npm installineng/(lockfile updated)npm run build✅npm run lint✅Co-authored-by: Copilot App 223556219+Copilot@users.noreply.github.com