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

Switch to vite #805

Merged
merged 28 commits into from
Dec 9, 2024
Merged

Switch to vite #805

merged 28 commits into from
Dec 9, 2024

Conversation

gjoseph
Copy link
Owner

@gjoseph gjoseph commented Dec 7, 2024

Addresses #574

@gjoseph gjoseph marked this pull request as ready for review December 9, 2024 05:56
@gjoseph
Copy link
Owner Author

gjoseph commented Dec 9, 2024

need storybook update

Gregory Joseph added 28 commits December 9, 2024 20:35
* https://www.darraghoriordan.com/2021/05/16/migrating-from-create-react-app-to-vite#1-update-your-packagejson
* Use vite-plugin-svgr 2.2.2 instead of 0.3.0 -- 2.3.0 brings compat with vite 4, so assuming this is for vite 3
npm -w tichu-web uninstall react-scripts
npm -w tichu-web uninstall @craco/craco
npm -w tichu-web install [email protected] [email protected] @vitejs/[email protected]
* remove %PUBLIC_URL%
* remove other now-irrelevant cruft
* Add script tag with the project entrypoint
* https://www.darraghoriordan.com/2021/05/16/migrating-from-create-react-app-to-vite#6-update-the-content-of-indexhtml
npm -w tichu-web install -D vitest jsdom
npm uninstall @types/jest jest ts-test # only the fist was explicitely depended on here
* vite: from 3.2.2 to 6.0.1
* vite-plugin-svgr: from 2.2.2 to 4.3.0
* @vitejs/plugin-react from 2.2.0 to 4.3.4
... which is part of create-react-app, which blows. It's not maintained. But I couldn't find a good and maintained
 alternative that didn't involved hand-configuring a lot of eslint

... it brings too many warnings and even errors that I'm not sure how/why passed through before. Perhaps from the
react 18 update recently.

... also brings back a ton of dependencies yuck

npm -w tichu-web install --save-dev [email protected] [email protected] [email protected] [email protected] [email protected]
…but not sure that's useful/wanted, since a) we have tslint?? b) it goes and check the transpiled js
…app by @storybook/react-vite

npm -w tichu-web uninstall --save-dev @storybook/react-webpack5 @storybook/preset-create-react-app
npm -w tichu-web install --save-dev @storybook/react-vite
@gjoseph
Copy link
Owner Author

gjoseph commented Dec 9, 2024

need storybook update

done

@gjoseph gjoseph enabled auto-merge December 9, 2024 09:38
Copy link

sonarqubecloud bot commented Dec 9, 2024

@gjoseph gjoseph mentioned this pull request Dec 9, 2024
@gjoseph gjoseph merged commit 8bc3831 into main Dec 9, 2024
10 checks passed
@gjoseph gjoseph deleted the switch-to-vite branch December 9, 2024 09:39
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.

1 participant