Skip to content

chore: switch package manager from pnpm to bun#1729

Open
lispking wants to merge 2 commits intofarion1231:mainfrom
lispking:main
Open

chore: switch package manager from pnpm to bun#1729
lispking wants to merge 2 commits intofarion1231:mainfrom
lispking:main

Conversation

@lispking
Copy link
Copy Markdown
Contributor

Copy link
Copy Markdown
Owner

@farion1231 farion1231 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why?

@lispking
Copy link
Copy Markdown
Contributor Author

The main motivation is performance and simpler tooling. Bun is npm-compatible, but installs dependencies much faster in practice, and it can also run scripts directly, so the overall dev setup becomes a bit more streamlined.

My thinking was: if the project does not rely on pnpm-specific behavior, using Bun could improve contributor onboarding and reduce install time locally and in CI.

That said, I understand package-manager choice is also about consistency and maintainability, so if pnpm is the preferred convention here, I’m okay with keeping it.

@farion1231
Copy link
Copy Markdown
Owner

The main motivation is performance and simpler tooling. Bun is npm-compatible, but installs dependencies much faster in practice, and it can also run scripts directly, so the overall dev setup becomes a bit more streamlined.

My thinking was: if the project does not rely on pnpm-specific behavior, using Bun could improve contributor onboarding and reduce install time locally and in CI.

That said, I understand package-manager choice is also about consistency and maintainability, so if pnpm is the preferred convention here, I’m okay with keeping it.

Thanks for your contribution! This is a significant change, so I’d like to take some time to evaluate it carefully before making a decision.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants