feat!: migrate to npm for better tooling#87
Merged
mohammadalfaiyazbitgo merged 1 commit intomasterfrom Jul 24, 2025
Merged
Conversation
mohammadalfaiyazbitgo
force-pushed
the
WP-5314/fix-semantic-versions
branch
2 times, most recently
from
ceceb47 to
7ec7f61
Compare
mohammadalfaiyazbitgo
force-pushed
the
WP-5314/fix-semantic-versions
branch
2 times, most recently
from
a8699b5 to
59b1531
Compare
There was a problem hiding this comment.
Pull Request Overview
This PR migrates the project from Yarn to npm package manager to follow semantic release GitHub Actions conventions. The change ensures consistency with the semantic release workflow requirements while maintaining all existing functionality.
Key changes:
- Migrated all build scripts and commands from Yarn to npm
- Updated lockfile references from yarn.lock to package-lock.json
- Changed package.json configuration from Yarn's "resolutions" to npm's "overrides"
Reviewed Changes
Copilot reviewed 6 out of 8 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| package.json | Updated build scripts to use npm and changed resolutions to overrides |
| README.md | Updated all command examples from yarn to npm run |
| Dockerfile | Changed dependency installation from yarn to npm ci |
| CLAUDE.md | Updated development command documentation to use npm |
| .github/workflows/release.yaml | Updated build and test commands for release workflow |
| .github/workflows/build-and-test.yaml | Updated all workflow steps to use npm instead of yarn |
| "scripts": { | ||
| "start": "nodemon", | ||
| "build": "yarn tsc --build --incremental --verbose . && cp package.json dist/", | ||
| "build": "npm run tsc -- --build --incremental --verbose . && cp package.json dist/", |
There was a problem hiding this comment.
[nitpick] The build script uses 'npm run tsc' but could be simplified to just 'tsc' since the tsc script is defined on the next line. Consider using 'tsc --build --incremental --verbose . && cp package.json dist/' for consistency and simplicity.
Suggested change
| "build": "npm run tsc -- --build --incremental --verbose . && cp package.json dist/", | |
| "build": "tsc --build --incremental --verbose . && cp package.json dist/", |
mohammadalfaiyazbitgo
changed the title
BREAKING CHAGE: Remove Yarn and Use npm Ticket: WP-5314
mohammadalfaiyazbitgo
force-pushed
the
WP-5314/fix-semantic-versions
branch
from
59b1531 to
1e96792
Compare
louib
approved these changes
Jul 24, 2025
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.
Ticket: WP-5314