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

Migrate from CRA to Vite #20

Draft
wants to merge 6 commits into
base: master
Choose a base branch
from
Draft

Conversation

neorth
Copy link
Contributor

@neorth neorth commented Oct 4, 2022

While looking into a crash I had trouble getting a good stack trace and the source maps didn't point to the correct location. So I thought, wth, lets migrate this to Vite.

Right now this is just a draft. A few things needs to be fixed:

  • Decide if this migration is wanted
  • Fix tests
  • Update readme
  • What else I forgot

@vercel
Copy link

vercel bot commented Oct 4, 2022

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

Name Status Preview Updated
o-scout ❌ Failed (Inspect) Oct 4, 2022 at 9:21PM (UTC)

@perliedman
Copy link
Owner

Hi Joakim! 👋 🙂

I have contemplated switching to Vite a few times but decided to put it further down on my todo. If you're interested in fixing the outstanding issues you mention above, I'd be 👍 for this change!

@neorth
Copy link
Contributor Author

neorth commented Oct 6, 2022

Great! I took a quick look at the tests and it looks like there will be som work getting jest to run when using vite, maybe it would be better to migrate to vitest while we're at it?

I don't really have an abundance of free time but I hope to get to it some evening ahead :)

@perliedman
Copy link
Owner

My github notifications is a mess, so saw this only now, sorry. But go for it! Haven't used vitest, but from the name it sounds like it makes sense if we're migrating to vite!

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