Conversation
✅ Deploy Preview for creative-fairy-df92c4 ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
| "dev": "buildc --deps-only -- wxt", | ||
| "build": "buildc --deps-only -- wxt build", | ||
| "build:all": "buildc --deps-only -- pnpm run --reporter-hide-prefix /^build:all:.*/", | ||
| "build:all": "buildc --deps-only -- bun run build:all:chrome-mv3 && bun run build:all:chrome-mv2 && bun run build:all:firefox-mv3 && bun run build:all:firefox-mv2", |
There was a problem hiding this comment.
Wish bun had a built-in way to do this.
There was a problem hiding this comment.
| '--prod', | ||
| '--json', | ||
| ]); | ||
| const { stdout } = await spawn('bun', ['why', 'esbuild', '--prod', '--json']); |
There was a problem hiding this comment.
TODO: Bun doesn't have a JSON output, load the lockfile instead.
Co-Authored-By: Okinea Dev <hi@okinea.dev>
|
This is a good idea, Bun is a way faster than pnpm :) |
Yeah, I'll add benchmarks once checks are passing. Which is going... slowly lol |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #2009 +/- ##
==========================================
+ Coverage 76.27% 79.15% +2.87%
==========================================
Files 113 125 +12
Lines 3031 3617 +586
Branches 680 809 +129
==========================================
+ Hits 2312 2863 +551
- Misses 636 669 +33
- Partials 83 85 +2 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
| pnpm pack | ||
| bun pm pack | ||
| sudo npm i -g npm@latest | ||
| /usr/local/bin/npm publish *.tgz |
There was a problem hiding this comment.
No that we don't have to run setup-node action, there is only one version of NPM installed and we don't need to specify the whole path.
| - name: Setup PNPM | ||
| uses: pnpm/action-setup@41ff72655975bd51cab0327fa583b6e92b6d3061 # v4.2.0 | ||
| with: | ||
| version: latest |
There was a problem hiding this comment.
Install the other package managers required for tests (was just bun, now it's just pnpm)
|
pkg.pr.new is failing due to stackblitz-labs/pkg.pr.new#458 |
|
Alright, pkg.pr.new is the last issue. |
3 participants
Overview
I've been migrating all my repos to use Bun. WXT is my last project that uses PNPM.
--ignore-scripts(no node_modules)--ignore-scripts(with node_modules)buildc all)Manual Testing
I've tested most dev scripts, but we'll probably have to fix a few things as they're used for the first time after merging this PR.
Related Issue