Skip to content

Migrate from yarn to npm#797

Open
Copilot wants to merge 4 commits intomainfrom
copilot/migrate-from-yarn-to-npm
Open

Migrate from yarn to npm#797
Copilot wants to merge 4 commits intomainfrom
copilot/migrate-from-yarn-to-npm

Conversation

Copy link
Contributor

Copilot AI commented Mar 23, 2026

The repository used yarn as its package manager. This replaces yarn with npm across all tooling and CI.

Changes

  • Lock file: Removed yarn.lock, generated package-lock.json via npm install
  • package.json: Added "lint": "eslint src" script
  • .github/workflows/pr-pipeline.yml: Replaced yarn commands with npm equivalents
  • build/pipeline.yml / build/pre-release.yml: Replaced yarn commands with npm equivalents

Command mapping

Before After
yarn install --frozen-lockfile npm ci
yarn run <script> npm run <script>
yarn eslint src npm run lint

💬 Send tasks to Copilot coding agent from Slack and Teams to turn conversations into code. Copilot posts an update in your thread when it's finished.

Copilot AI linked an issue Mar 23, 2026 that may be closed by this pull request
Co-authored-by: rzhao271 <7199958+rzhao271@users.noreply.github.com>
Agent-Logs-Url: https://github.com/microsoft/vscode-livepreview/sessions/bfe2ff3f-b8f6-4571-b816-f0db4d5f396b
Copilot AI changed the title [WIP] Migrate project from yarn to npm Migrate from yarn to npm Mar 23, 2026
Copilot AI requested a review from rzhao271 March 23, 2026 16:09
@rzhao271
Copy link
Contributor

@copilot Check that eslint is a devdependency now and regenerate the lock file

Copy link
Contributor Author

Copilot AI commented Mar 23, 2026

@copilot Check that eslint is a devdependency now and regenerate the lock file

eslint is indeed a devDependency in package.json (along with @typescript-eslint/eslint-plugin, @typescript-eslint/parser, eslint-plugin-header, etc.). The package-lock.json was generated via npm install in commit f38afad and is already up to date — re-running npm install produces no changes to the lock file.

@rzhao271
Copy link
Contributor

@copilot Resolve merge conflict

Copy link
Contributor Author

Copilot AI commented Mar 23, 2026

@copilot Resolve merge conflict

Done in 85fd52e. Merged origin/main (ESLint 9 flat config, #798) into this branch. The only conflict was yarn.lock — kept it deleted. Also removed package-lock.json from .gitignore (main had added it, but we want to track it), and regenerated package-lock.json with the updated ESLint 9 dependencies.

@rzhao271 rzhao271 marked this pull request as ready for review March 23, 2026 18:48
@rzhao271
Copy link
Contributor

@copilot Do not use npx. Add new lint script.

Copy link
Contributor Author

Copilot AI commented Mar 23, 2026

@copilot Do not use npx. Add new lint script.

Done in eae4b38. Added "lint": "eslint src" to package.json scripts and updated all CI files to use npm run lint instead of npx eslint src.

@rzhao271 rzhao271 enabled auto-merge (squash) March 23, 2026 18:51
Copilot AI requested a review from rzhao271 March 23, 2026 18:51
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.

Migrate from yarn to npm

2 participants