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

Create React App -> Vite #851

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Create React App -> Vite #851

wants to merge 2 commits into from

Conversation

lowtorola
Copy link
Contributor

@lowtorola lowtorola commented Nov 18, 2024

Closes #842 🚀

  • Uninstalled CRA
  • Installed vite 🤩
  • Removed all frontend2 testing references (we don't have tests lol)
  • Removed unnecessary testing GitHub Action
  • bumped node version for vite compatibility
  • bumped eslint version for vite compatibility
  • rebuilt eslint testing for new "flat config" eslint version (NOT WORKING 😢 )

Side note: it should be really easy to do relative imports now! I may look at that when i get a chance!

@lowtorola
Copy link
Contributor Author

@acrantel this almost works it just needs its upgraded eslint to have the rules that the old eslint had i think? was there a systematic way we determined those old rules or some way I could know what exactly we want to have throw an error when running lint?

eslint almost working need rules
ignore magic 0 1 -1

tmp checkpoint commit

fixed all lint errors

rebased
@lowtorola
Copy link
Contributor Author

@acrantel this is done 🚀

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.

Create React App -> Vite
1 participant