-
-
Notifications
You must be signed in to change notification settings - Fork 78
chore: Migrate to vite #2928
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
chore: Migrate to vite #2928
Conversation
Signed-off-by: Louis Chemineau <louis@chmn.me>
Signed-off-by: Louis Chemineau <louis@chmn.me>
844b366 to
dc13cc5
Compare
Welcome to Codecov 🎉Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests. Thanks for integrating Codecov - We've got you covered ☂️ |
22c1cbd to
db43d2f
Compare
Signed-off-by: Louis Chemineau <louis@chmn.me>
Signed-off-by: Louis Chemineau <louis@chmn.me>
Signed-off-by: Louis Chemineau <louis@chmn.me>
757ffa3 to
42c1565
Compare
Signed-off-by: Louis Chemineau <louis@chmn.me>
42c1565 to
5b64e48
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
not needed with vite anymore (as you also replaced jest with vitest)
| @@ -90,29 +91,24 @@ | |||
| "@nextcloud/cypress": "^1.0.0-beta.12", | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
babel config above not needed anymore
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
(if cypress is also using vite)
Minimal changes to switch to vite.
Better reviewed per commit.