Skip to content

Add override for serialize-javascript#1261

Merged
rzhao271 merged 3 commits into
mainfrom
copilot/add-override-for-serialize-javascript
Apr 15, 2026
Merged

Add override for serialize-javascript#1261
rzhao271 merged 3 commits into
mainfrom
copilot/add-override-for-serialize-javascript

Conversation

Copilot AI commented Apr 14, 2026

Copy link
Copy Markdown
Contributor
  • Review new PR comment requesting lockfile refresh
  • Run npm i to update lockfile
  • Validate resulting dependency state with targeted checks
  • Run final validation checks
  • Reply to PR comment with commit hash

Copilot AI requested a review from rzhao271 April 14, 2026 23:09
@rzhao271

Copy link
Copy Markdown
Collaborator

@copilot The override should be to version 7.x

Copilot AI commented Apr 15, 2026

Copy link
Copy Markdown
Contributor Author

@copilot The override should be to version 7.x

Updated in commit 63a933b: the root override now pins serialize-javascript to 7.x in package.json.

@rzhao271

Copy link
Copy Markdown
Collaborator

@copilot Run npm i to update the lockfile.

Copilot AI commented Apr 15, 2026

Copy link
Copy Markdown
Contributor Author

@copilot Run npm i to update the lockfile.

Done in commit da40085. I ran npm i and committed the updated package-lock.json so the serialize-javascript 7.x override is reflected there. No UI changes in this update.

@rzhao271
rzhao271 marked this pull request as ready for review April 15, 2026 17:27
@rzhao271
rzhao271 enabled auto-merge (squash) April 15, 2026 17:27
@rzhao271 rzhao271 added this to the 1.117.0 milestone Apr 15, 2026
@rzhao271
rzhao271 merged commit 165b0f2 into main Apr 15, 2026
5 checks passed
@rzhao271
rzhao271 deleted the copilot/add-override-for-serialize-javascript branch April 15, 2026 18:47
adrianstephens pushed a commit to adrianstephens/vscode-vsce that referenced this pull request Jun 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants