Skip to content

chore: bump node-pty@1.1.0-beta35 - #267939

Merged
Robo (deepak1556) merged 1 commit into
mainfrom
robo/bump_node_pty
Sep 23, 2025
Merged

Robo (deepak1556) merged 1 commit into
mainfrom
robo/bump_node_pty

Conversation

@deepak1556

Copy link
Copy Markdown
Collaborator

@deepak1556 Robo (deepak1556) added this to the September 2025 milestone Sep 23, 2025
@deepak1556 Robo (deepak1556) self-assigned this Sep 23, 2025
@deepak1556
Robo (deepak1556) marked this pull request as ready for review September 23, 2025 12:01
Copilot AI review requested due to automatic review settings September 23, 2025 12:01
@deepak1556
Robo (deepak1556) enabled auto-merge (squash) September 23, 2025 12:01

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 updates the node-pty dependency from version 1.1.0-beta33 to 1.1.0-beta35 across the VS Code codebase. The update is related to upstream changes in the node-pty repository as referenced in PR #801.

Key changes:

  • Bumps node-pty dependency version in both main and remote package configurations
  • Removes the caret (^) prefix from the version specification, pinning to the exact beta version

Reviewed Changes

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

File Description
package.json Updates node-pty dependency to exact version 1.1.0-beta35
remote/package.json Updates node-pty dependency to exact version 1.1.0-beta35 for remote scenarios
Files not reviewed (1)
  • remote/package-lock.json: Language not supported

@deepak1556
Robo (deepak1556) merged commit a26c8fc into main Sep 23, 2025
28 checks passed
@deepak1556
Robo (deepak1556) deleted the robo/bump_node_pty branch September 23, 2025 12:04
@xiaoxiangmoe

ZHAO Jin-Xiang (xiaoxiangmoe) commented Sep 23, 2025 •

Copy link
Copy Markdown
Contributor

Robo (@deepak1556) You should also add node-gyp as devDependencies, because npx --offline node-gyp need it to realize lock node-gyp used for compile commands.
If you don't add it, npx may try to download newer version of node-gyp.

@xiaoxiangmoe

Copy link
Copy Markdown
Contributor

I create a PR to do this

https://github.com/microsoft/vscode/pull/267663

@vs-code-engineering vs-code-engineering Bot locked and limited conversation to collaborators Nov 7, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants