-
Notifications
You must be signed in to change notification settings - Fork 167
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
Add Vite function #435
base: master
Are you sure you want to change the base?
Add Vite function #435
Conversation
This adds the ability to use the Laravel Vite helper
Should we add this to the config, perhaps default commented out? |
@barryvdh Definitely one of those! Is there any significant overhead with turning it on by default? Maybe we can leave it off to be on the safe side. Let ke know if you have a preference. |
@barryvdh Added it in there commented out. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, useful for our migration to Vite
twig + vite integration done needs rcrowe/TwigBridge#435 hmr: to be tested
@barryvdh do you have time to take a look at this? thanks! |
I would like to see this too |
@barryvdh Anything else needed from me on this one? |
This adds the ability to use the Laravel Vite helper