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

(internal): pnpm migration #4261

Merged
merged 28 commits into from
Aug 12, 2024
Merged

(internal): pnpm migration #4261

merged 28 commits into from
Aug 12, 2024

Conversation

dsinghvi
Copy link
Member

@dsinghvi dsinghvi commented Aug 11, 2024

This PR migrates fern to use pnpm instead of yarn. Couple changes:

  • Commands must be prefixed with pnpm (e.g. pnpm install).
  • To filter to a workspace you can run pnpm --filter <package-name> comamnd
  • About 50% of workspaces are now using vitest
  • mrlint is now deleted and completely removed from the repo (we will have to manage tsconfigs on our own)
  • pnpm test runs all unit tests and is more performant (you should be able to run these locally).

@dsinghvi dsinghvi marked this pull request as ready for review August 11, 2024 19:37
@dsinghvi dsinghvi force-pushed the dsinghvi/pnpm-migrations branch 2 times, most recently from aee6f0e to dedfe47 Compare August 11, 2024 22:01
@dsinghvi dsinghvi force-pushed the dsinghvi/pnpm-migrations branch from ca56ff3 to 37a8157 Compare August 12, 2024 00:51
@dsinghvi dsinghvi changed the title pnpm migration (internal): pnpm migration Aug 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant