Closed ooystein closed 1 year ago
I have tested to link pacakge localy to compare the new build and it seems to work fine. But I encourage reviewers to test this PR against existing projects dependent on this library.
I have tested to link pacakge localy to compare the new build and it seems to work fine. But I encourage reviewers to test this PR against existing projects dependent on this library.
Also tested linking these changes for our usage. Seems to work just fine 🎉
Fixed review comments
To simplify setup even further we should consider moving tests from jest to vitest. Then we can get rid of babel and jest configurations and dependencies. If vitest requires just as much configuration it might not be worth it, but if we can use vite for the tests and everything is configured the same it would make the whole setup much simpler to maintain.