chore: migrate to pnpm#31
Conversation
danielmrdjanov
changed the title
There was a problem hiding this comment.
Pull request overview
Migrates the repository from Yarn/npm usage to pnpm across developer docs, scripts, and CI/CD so dependency installs and publishing align with pnpm workflows.
Changes:
- Remove
yarn.lockand declare pnpm as the package manager inpackage.json. - Update scripts/docs to use pnpm commands for common tasks (install/build/generate/lint/test).
- Switch GitHub Actions workflows (PR + release) to install/build/publish with pnpm.
Reviewed changes
Copilot reviewed 6 out of 9 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
yarn.lock |
Removed Yarn v1 lockfile as part of pnpm migration. |
scripts/generate-typescript-model.sh |
Updates model-generation instructions and invocation away from Yarn. |
package.json |
Declares pnpm via packageManager and updates build script / dev deps. |
README.md |
Updates install instructions to pnpm. |
CONTRIBUTING.md |
Updates contributor commands to pnpm equivalents. |
.github/workflows/release.yaml |
Updates release pipeline to set up pnpm, install with frozen lockfile, build and publish with pnpm. |
.github/workflows/pr.yml |
Updates PR CI to install/lint/test using pnpm. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 6 out of 9 changed files in this pull request and generated 2 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 7 out of 10 changed files in this pull request and generated 2 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 7 out of 10 changed files in this pull request and generated 3 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 7 out of 10 changed files in this pull request and generated 2 comments.
Comments suppressed due to low confidence (2)
.github/workflows/release.yaml:46
- The section separators (
######/## ...) are not YAML comments because they don’t start with#. These bare scalars at this indentation level make the workflow file invalid YAML. Prefix these lines with#(or remove them) so the workflow can be parsed.
######
## Calculate next version based on user semver input
## Update version and create tag and push to main
######
.github/workflows/release.yaml:108
- Same as above: the
######/## ...lines here need to be YAML comments (start with#) or removed; otherwise the workflow won’t parse.
######
## Create GitHub release and release notes
######
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 7 out of 10 changed files in this pull request and generated 3 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 7 out of 10 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
… adjusting dependency management settings
…s since its accepts minutes
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 9 out of 12 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
No description provided.