Skip to content
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

Migrate to yarn package manager #3723

Closed
3 tasks done
Tracked by #4580
rak-phillip opened this issue Jan 9, 2023 · 2 comments · Fixed by #5106
Closed
3 tasks done
Tracked by #4580

Migrate to yarn package manager #3723

rak-phillip opened this issue Jan 9, 2023 · 2 comments · Fixed by #5106
Labels
component/ci Stories tied to CI for automation component/ui kind/quality quality improvements, refactoring, Automation via CI, E2E, Integration, CLI or REST API

Comments

@rak-phillip
Copy link
Contributor

rak-phillip commented Jan 9, 2023

Rancher Dashboard and Shell both depend on yarn. Areas that will need to be addressed

  • Resolve build warnings/error that arise as a result of the change of tooling
  • Update developer documentation
  • Update CI/CD steps

Additionally, please file an issue to move to yarn (if @rancher/shell is already using it) so that we match tools.

Originally posted by @mook-as in #3696 (review)

@rak-phillip rak-phillip added component/ci Stories tied to CI for automation component/ui kind/quality quality improvements, refactoring, Automation via CI, E2E, Integration, CLI or REST API labels Jan 9, 2023
@RihanArfan
Copy link

Out of curiosity, how comes Yarn and not pnpm?

@rak-phillip
Copy link
Contributor Author

A large motivator for moving to yarn is to keep our tooling aligned with Rancher Manager.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component/ci Stories tied to CI for automation component/ui kind/quality quality improvements, refactoring, Automation via CI, E2E, Integration, CLI or REST API
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants