-
-
Notifications
You must be signed in to change notification settings - Fork 2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
build: migrate from yarn to pnpm, remove CircleCI config #4670
Conversation
✅ Deploy Preview for ngrx-io ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
a132900
to
891ee29
Compare
891ee29
to
968adbb
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🤝
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good! 🚀
I searched for yarn
and found a couple more places that also need to be updated, I'll update those tomorrow.
Additionally, the main workflow needs also need to include a release step.
When that's done we can close #4507 🚀
PR Checklist
Please check if your PR fulfills the following requirements:
PR Type
What kind of change does this PR introduce?
What is the current behavior?
yarn
1.x is removed in favor ofpnpm
for dependency installation and running commands. The docs-app still usesyarn
but will be replaced with the new docs site.Closes #
What is the new behavior?
Does this PR introduce a breaking change?
Other information