-
Notifications
You must be signed in to change notification settings - Fork 103
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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
Adds Bun support #392
base: main
Are you sure you want to change the base?
Adds Bun support #392
Conversation
It will be great to see the Bun support! |
@vitalyliber left to fix tests ;( but you can use my pr to confirm at least this "fast" workaround works for you too |
46d9c33
to
2613df8
Compare
@ElMassimo please review this PR Notes:This is a very naive approach to start using it. I already use it for my projects. |
I found some strange issue: |
Hello, @pftg Currently, bun does not support brotli: oven-sh/bun#267 ViteRuby could work with bun, but we have vite-plugin-rails, which depends on brotli compression currently, may be we should wait some time before brotli will be implemented or conditionally enable brotli in plugin? @ElMassimo any thoughts on this? |
@vladimirtemnikov Yeah, you are right. I had to disable brotli by the way, to use this version. So if this is a hard requirement to support the compress plugin, then there is a workaround. So, for now, I use I think better to start using without brotli to find more issues |
2613df8
to
f3bce48
Compare
@vladimirtemnikov @ElMassimo updated PR, what do you think to add notice in the Readme that it's not compatible with the vite rails plugin and list of supported bun versions? |
Hmm, for development mode, everything was fine, but as soon as I tried to run on Heroku with production mode, the compilation got stuck. I found that when removing RubyPlugin, it's not stuck. I need help to set env to see the real problem. I'll try to look into it later. Will be great to find a guide on how to set up development env for those plugins with bun and with integration testing in the real app. |
Any ideas to bring this forward? Also rails 7.1. has now bun support. Would be nice to when it can also be used in the vite-ruby/rails context. |
Yeah need to check latest tests, maybe now it's working. For now I'm using this approach to use bun https://jtway.co/integrating-bun-with-vite-ruby-for-lightning-fast-frontend-builds-bbd220fefdc0 |
5892ef2
to
d044bd6
Compare
@ElMassimo, with the latest bun, it looks like it works. I'm working on the cleanup in a separate PR jetthoughts#2 |
https://github.com/ElMassimo/vite_ruby/actions/runs/8186414098/job/22384757347?pr=392 - it's a new issue that appeared lately, I will check, but I believe I need to rebase |
89b2388
to
412a06a
Compare
412a06a
to
94f4ec0
Compare
@pftg all green - great job! |
Description 馃摉
Adds support of Bun
Background 馃摐
Lately, Bun has adopted Vite, so we can use it for compilation assets.