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: upgrade Remix + Vite #222

Merged
merged 1 commit into from
Jan 2, 2023
Merged

feat: upgrade Remix + Vite #222

merged 1 commit into from
Jan 2, 2023

Conversation

ruisaraiva19
Copy link
Contributor

@ruisaraiva19 ruisaraiva19 commented Dec 18, 2022

This upgrades the following:

Remix

From version 1.7.6 to version 1.9.0.

All Remix imports are updated to use specific packages in favor of the deprecated remix package. This removes the need to the setup-remix script as well.

All Remix apps were also updated to use the latest template configuration options for remix.config.js files.

dev & build scripts

From tsc for dev + vite build for build to tsup for both.

The bundling process was upgraded to use an identical approach for both dev and build scripts.
Previously, it was using tsc for the dev script and vite for the build script.
Now, it's using tsup (esbuild based) to bundle packages for both dev and build scripts.

Vite

From version 2.9.x to version 4.0.x.

@vercel
Copy link

vercel bot commented Dec 18, 2022

Someone is attempting to deploy a commit to a Personal Account owned by @airjp73 on Vercel.

@airjp73 first needs to authorize it.

@vercel
Copy link

vercel bot commented Jan 2, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
remix-validated-form ✅ Ready (Inspect) Visit Preview Jan 2, 2023 at 2:47PM (UTC)

@airjp73
Copy link
Owner

airjp73 commented Jan 2, 2023

Great work! Thanks for putting in the time to do this.

@airjp73 airjp73 merged commit 996e08a into airjp73:main Jan 2, 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.

2 participants