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

feat: Add bun documentation to README.md #918

Closed
wants to merge 1 commit into from

Conversation

worm-emoji
Copy link

@worm-emoji worm-emoji commented Jul 3, 2024

Changes being requested

  • Add bun install information to README.

Additional context & links

I noticed that the readme had install instructions for npm and Deno. I figured it would be helpful to have Bun as well.

I understand that there may be some hesitation to add another download method, but Bun has more monthly installations than Deno, so if there's only room for one other non-npm option, it might make sense to keep Bun and drop Deno.

@worm-emoji worm-emoji requested a review from a team as a code owner July 3, 2024 01:22
@rattrayalex
Copy link
Collaborator

Thanks for the contribution! We ❤️ bun, but its installation is intuitive to any bun user, and not worth the extra installation instruction.

@rattrayalex rattrayalex closed this Jul 6, 2024
@Jarred-Sumner
Copy link

Sad

@worm-emoji
Copy link
Author

worm-emoji commented Jul 7, 2024

Plenty of projects list support for a combination of npm/Bun/Deno. Here, Bun is punished for being more "intuitive" to use?

Feels like an unrigorous way to establish inclusion. Please reconsider, not to mention that adding Bun docs could be a great way to get exposure for your package.

@SlexAxton
Copy link

came here for the team reunion

@SlexAxton
Copy link

hope everyone is doin good

@rattrayalex
Copy link
Collaborator

Hah. Hey guys, good to see you! Luke, missed that was you – great avatar.

I'll float this with the team…

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.

None yet

4 participants