Repository navigation
refactor(*): removing dependency on yarn - #1885
Conversation
|
One or more custom setup steps configured for this repository failed during this Copilot code review run: Setup steps run before each review. If the review above is missing context, or no review was posted at all, the failing step above may be the cause. See the workflow run for failure details, fix your setup steps configuration, and re-request a review. Note You can configure setup steps for Copilot code review separately from Copilot cloud agent with a |
There was a problem hiding this comment.
🟢 Approval recommended
The Yarn-to-npm workflow migration has no active review issues.
No open issues 🎉
What changed in this PR
This PR migrates the monorepo’s dependency-management workflow from Yarn to npm, aligning CI, release configuration, developer guidance, and CLI messages.
Changes:
- Switch installation and caching in CI to npm.
- Update package configuration for npm overrides, install-script permissions, and release-time lockfile refresh.
- Replace Yarn instructions in documentation, maintenance skills, and diagnostics; update the corresponding test.
| File | Description |
|---|---|
spec/unit/mcp-cli-spec.ts |
Expect npm installation guidance in the MCP error message. |
scripts/build-sbom.ts |
Update installation guidance and clarify a dev-tree comment. |
README.md |
Document npm workspaces and installation steps. |
packages/cli/lib/commands/mcp.ts |
Recommend npm when the MCP package is missing. |
package.json |
Add npm overrides, script permissions, and release lockfile refresh. |
lerna.json |
Set npm as Lerna’s package client. |
AGENTS.md |
Align contributor guidance with npm workflows. |
.gitignore |
Allow tracking the root npm lockfile; remove the Yarn log pattern. |
.github/workflows/npm-publish.yml |
Install build dependencies with npm ci. |
.github/workflows/nodejs.yml |
Use npm caching, installation, and script commands. |
.github/workflows/copilot-setup-steps.yml |
Prepare agent environments with npm. |
.github/workflows/build-docs-db.yml |
Use npm caching and installation for database builds. |
.github/actions/build-framework-docs/action.yml |
Use npm for framework documentation dependencies. |
.agents/skills/igniteui-cli-template-version-update/SKILL.md |
Document npm dependency installation. |
.agents/skills/igniteui-cli-template-version-update/scripts/template-pins.mjs |
Recommend npm ci when semver is unavailable. |
.agents/skills/igniteui-cli-smoke-test/SKILL.md |
Update installation and lockfile-handling guidance. |
Review effort: Balanced
|
coverage: 95.666% (+0.003%) from 95.663% — feature/removing-yarn into master |
|
One or more custom setup steps configured for this repository failed during this Copilot code review run: Setup steps run before each review. If the review above is missing context, or no review was posted at all, the failing step above may be the cause. See the workflow run for failure details, fix your setup steps configuration, and re-request a review. Note You can configure setup steps for Copilot code review separately from Copilot cloud agent with a |
Co-authored-by: Damyan Petev <damyanpetev@users.noreply.github.com>
Co-authored-by: Damyan Petev <damyanpetev@users.noreply.github.com>
kdinev
changed the title
Description
Related Issue
Closes #
Type of Change
Affected Packages
igniteui-cli(packages/cli)@igniteui/cli-core(packages/core)@igniteui/angular-templates(packages/igx-templates)@igniteui/angular-schematics(packages/ng-schematics)@igniteui/mcp-server(packages/igniteui-mcp)Checklist
npm run test)npm run build)npm run lint)Additional Context