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

RC/Bun - Testing bun features as package manager and runtime for projects as a default solution #59

Merged
merged 7 commits into from
Apr 11, 2024

Conversation

devmozao
Copy link
Collaborator

All script commands are working as expected, but bun is installed per homebrew. I'm running with win/ws2 with debian.

I tried to install bun locally on the project and run commands from there - in order to force the user to use bun, but without giving him the effort to install bun itself and ease the learning curve - but got some conflicts when running command scripts.

Other point that didn't work as expected: the --bun flag.

When I run a command script as bun run --bun dev, it works as expected. When I put the --bun flag inside the script, let's say "dev": "--bun vite --host", and run bun dev, it doesn't work as expected, unfortunatelly. I think we should be able to use the flag on the scripts, so on your daily job, all people that works on the project, doesn't have to remember to have to use the flag, so it's a better DX overall. Also putting the flag at the end of the flag, also doesn't do.

Besides that: works like a charm.

@raisiqueira
Copy link
Member

CI running in 16s 🤯

CleanShot 2024-01-26 at 17 55 12

@raisiqueira
Copy link
Member

LFGT 🚀

@devmozao
Copy link
Collaborator Author

updated deps to latest. everything working as expected.

@luigieai
Copy link
Member

Just solved a quick merge conflict, besides that

LGTM!

@devmozao
Copy link
Collaborator Author

🚀 🌋 💥 🐰

@devmozao devmozao merged commit 8364667 into main Apr 11, 2024
1 check passed
@devmozao devmozao deleted the rc/bun branch April 11, 2024 15:16
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.

3 participants