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

fix: Update install method for Bun #64151

Merged
merged 7 commits into from
Apr 7, 2024
Merged

fix: Update install method for Bun #64151

merged 7 commits into from
Apr 7, 2024

Conversation

Zaid-maker
Copy link
Contributor

@Zaid-maker Zaid-maker commented Apr 6, 2024

What?

So, i was trying to create a Next.js project with Bun as i was experimenting with it and i got error creating the project error: could not determine executable to run for package create so i added - after create and works and created by project successfully.

Before

image

After

image

@Zaid-maker Zaid-maker requested review from a team as code owners April 6, 2024 20:56
@Zaid-maker Zaid-maker requested review from ismaelrumzan and StephDietz and removed request for a team April 6, 2024 20:56
Copy link
Member

@ijjk ijjk left a comment

Choose a reason for hiding this comment

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

See #64081 seems this is an issue in bun as either variation of this command is supposed to work according to their docs

@Zaid-maker
Copy link
Contributor Author

Zaid-maker commented Apr 6, 2024

See #64081 seems this is an issue in bun as either variation of this command is supposed to work according to their docs

I double check by upgrading my Bun and the issue is still same.

@Zaid-maker
Copy link
Contributor Author

Verified on a Linux Machine as well!

image

@Zaid-maker
Copy link
Contributor Author

Zaid-maker commented Apr 6, 2024

I read the docs they say both of these work identically might a bug in Bun idk!! Tested on Windows and Linux tho.

The Guide use the create next-app Hmmm

@Zaid-maker
Copy link
Contributor Author

I have update to show to use both methods... Sounds good?

@Zaid-maker Zaid-maker changed the title fix: Typo in install method for Bun fix: Update install method for Bun Apr 7, 2024
@Zaid-maker
Copy link
Contributor Author

@ijjk Is it good now? 😄

@ijjk
Copy link
Member

ijjk commented Apr 7, 2024

Let's leave it as this as from testing against latest bun this appears to work correctly while the other doesn't, thanks!

@ijjk ijjk merged commit a42efae into vercel:canary Apr 7, 2024
32 checks passed
@Zaid-maker Zaid-maker deleted the patch-1 branch April 7, 2024 17:06
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Apr 22, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants