-
-
Notifications
You must be signed in to change notification settings - Fork 431
chore: Use Bun for package management #1649
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
Conversation
✅ Deploy Preview for creative-fairy-df92c4 ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
@wxt-dev/analytics
@wxt-dev/auto-icons
@wxt-dev/browser
@wxt-dev/i18n
@wxt-dev/module-react
@wxt-dev/module-solid
@wxt-dev/module-svelte
@wxt-dev/module-vue
@wxt-dev/storage
@wxt-dev/unocss
@wxt-dev/webextension-polyfill
wxt
commit: |
| pnpm-lock.yaml linguist-generated | ||
| package-lock.json linguist-generated | ||
| bun.lock linguist-generated | ||
| bun.lockb linguist-generated |
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.
| bun.lockb linguist-generated |
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.
There's still a test fixture containing a binary lockfile, I'll try and remove this one when I update E2E tests to use bun.
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.
There's still a test fixture containing a binary lockfile, I'll try and remove this one when I update E2E tests to use bun.
Okay
|
After merging this PR, we can then migrate all It's not urgent, everything is drop-in replacement there except mocks, in Bun it's a bit specific |
Co-authored-by: Okinea Dev <[email protected]>
Yeah, if we decide to migrate tests, we'll do that in a separate PR. |
|
Please add |
|
Errors in tests due to lack of pnpm? It seems that Node.js and pnpm should be added to the test workflows |
Yeah, boarding a plane rn and traveling all day today, so this will have to wait until tomorrow. |
|
Is there any progress now? |
|
We'll I added provenence the other day because of NPM's changes to tokens. Might try to update this soon. Got a lot of PRs to get through after I took my break, and this isn't a top priority rn. |
|
This PR is out-of-date, closing in favor of #2009 |
|
@okineadev I'll make sure to add |
This closes #1643.