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: support Vite 5 #45

Merged
merged 11 commits into from
Dec 11, 2023
Merged

feat: support Vite 5 #45

merged 11 commits into from
Dec 11, 2023

Conversation

wyatt-herkamp
Copy link
Contributor

It is not working on Legacy Browsers the tests for Legacy browser is failing. If you know how to fix this please help.

I did updated the other dependencies.

@babel/preset-env wanted a peer dependency of @babel/core

@aljazerzen
Copy link

aljazerzen commented Dec 5, 2023

@Menci Can this be merged?

FYI: legacy browser tests fail on my machine as well

@Menci
Copy link
Owner

Menci commented Dec 9, 2023

Thanks. I'll check it soon.

@wyatt-herkamp
Copy link
Contributor Author

I propose we have a new major release and drop Vite 2/3. And Node 14/16.

@Menci
Copy link
Owner

Menci commented Dec 9, 2023

@aljazerzen @wyatt-herkamp Now everything should just work. Please check it and after then I'll merge and release it.

For the broken test, it's because the upgrade of playwright. I'm not sure it's the issue of playwright or corresponding version of Firefox that simulation of a legacy browser with dom.moduleScripts.enabled isn't working. Just pinning playwright's version does the fix.

@Menci Menci changed the title Attempt at updating to Vite 5 feat: support Vite 5 Dec 9, 2023
@wyatt-herkamp
Copy link
Contributor Author

Tested against my website it worked fine. Test Suite passed too.

@Menci Menci merged commit 9d500ce into Menci:main Dec 11, 2023
4 checks passed
@Menci Menci mentioned this pull request Dec 22, 2023
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