-
Notifications
You must be signed in to change notification settings - Fork 68
[wip] test bun migration #586
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
base: main
Are you sure you want to change the base?
Changes from all commits
48444c1
ec5ba64
5c7ae55
9657f35
0f93e12
b0ccc79
cd66ef3
deb34e3
10d743c
7b604e2
3dead8b
862dbb8
c6b5dd8
6ade54b
7593e89
eaea18c
5200266
497bb1e
9d1fd6d
9b0a7d1
8867da2
d5245b1
0d9bcc2
f724463
f8d3be7
783610a
05fc8b5
575bbd7
b713bc0
c5ffe28
7f64983
aead98a
972a647
5d51935
c431239
9c086e2
b7b18c5
605053a
af06dbb
5fcdfaf
6c37743
7cd59c0
12f7c22
3c655fc
f6b01ab
03fa6b4
fd5e1bb
0f0c860
d8917b5
836af07
d8b19a4
f82e1a2
0aa45af
3bb9e7e
5755609
03a5291
98a7503
5fc9697
4dbd615
dd5d03a
cda5dff
7545c16
549e5ac
315fc93
485dc6e
9ee2aad
e2581d3
059c7cb
3f0ef7c
3a1cf58
bf6fa69
72e7c3c
a918b0e
17e98cf
5acebd9
0ab8357
09df8b4
bdb0f66
5a061c2
8f1b678
84c3ddc
9ccc20e
0196f88
49f7953
328fc41
55b0423
1f1773a
2b6f3a9
6592096
9672927
c6a73fc
495b91d
0b85c6e
34f77a2
2ddd110
4475cd8
a082a34
66e0ca9
7b1cd0b
bd1762c
37b5524
24c8a4f
2d9f9dc
7f31ca3
d493db4
70641b1
c30b544
f19723e
e43e0da
59d6d4a
2f77fe4
9f73c64
f1445ff
3c761b7
3ce1726
f7a5642
e7e0ee0
839b31a
9583bec
27c8251
b7fcc20
f408e7b
3b3f58d
27fd604
e22aa27
009fce0
94ca937
8691a47
687f79f
eabac11
381ad0b
7214173
e485d09
23c8248
a627124
adb532b
3cd0953
b6e7c86
ea0afdd
3d1117c
162bf4a
4c0cd2e
ed1bb5d
2b2183d
58eaeb2
e788b6d
e1b3981
f5e249d
1b75ba9
6ad15c8
57c2e54
825d909
e60dfc8
901156f
c1162ea
cb4fe34
60962f0
a74af5c
380f4ab
a23d46f
4f33938
c6c0275
407a7f9
4a5c6a0
933cae9
f633bcb
8563e0b
57638bc
49574e8
2a8c877
e34e41f
c33bcfc
bdaea50
811fdcb
6790601
cdafb14
a18d45a
9228c79
432828e
b64c75a
15d45e2
ec43d7c
f1cf00a
59bf854
415d0f3
a95f4bd
4f8a366
d10017b
bc07dd7
259b5bb
61b0772
92da710
08148ab
366203e
e6b066e
db87852
a6a4735
780dac2
1864dda
504f2b0
a57c4df
5f9fbce
b40d68e
b52bab9
f9d67c4
600b5fe
5ea8049
40ce9d1
5e36ff3
ed6fb66
ec32f68
bbb0e20
3628b87
15352f7
5bdaf23
ec3564c
78e48b0
c6b670f
6fc2a4d
f235a56
fd56768
3db366f
b51766a
94db4f0
81fb1b8
dc3b79c
3911291
10d7b2e
9e72b02
e98d179
88b2650
907567e
46a4881
18b7ef2
569253a
3d6dbb1
8681cab
17ac087
bed9353
9309e5e
b807d43
a87ea01
930bdff
10f5a6c
56ddce7
4b62e8c
7331006
1017fec
ee7e045
4510d69
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,8 +1,7 @@ | ||
| ### Release Note | ||
|
|
||
| - [ ] No release needed (docs/chore/test-only/private package) | ||
| - [ ] Changeset added via `pnpm changeset` (select packages + bump) | ||
| - [ ] Changeset added via `bun x changeset` (select packages + bump) | ||
| - Bump type: Patch / Minor / Major (choose patch for all if unsure) | ||
| - Packages: ... | ||
|
|
||
|
|
||
| Original file line number | Diff line number | Diff line change | ||||||||||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
|
|
@@ -26,18 +26,16 @@ jobs: | |||||||||||||||||||||||
| with: | ||||||||||||||||||||||||
| fetch-depth: 0 # Needed for last updated dates | ||||||||||||||||||||||||
|
|
||||||||||||||||||||||||
| - name: Setup Node.js | ||||||||||||||||||||||||
| uses: actions/setup-node@v4 | ||||||||||||||||||||||||
| - name: Set up Bun | ||||||||||||||||||||||||
| uses: oven-sh/setup-bun@v2 | ||||||||||||||||||||||||
| with: | ||||||||||||||||||||||||
| node-version: '20' | ||||||||||||||||||||||||
| cache: npm | ||||||||||||||||||||||||
| cache-dependency-path: ./docs/package-lock.json | ||||||||||||||||||||||||
| bun-version: '1.3.5' | ||||||||||||||||||||||||
|
Comment on lines
+29
to
+32
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Same Align with Proposed fix - name: Set up Bun
- uses: oven-sh/setup-bun@v1
+ uses: oven-sh/setup-bun@v2
with:
bun-version: '1.3.5'📝 Committable suggestion
Suggested change
🤖 Prompt for AI Agents |
||||||||||||||||||||||||
|
|
||||||||||||||||||||||||
| - name: Install dependencies | ||||||||||||||||||||||||
| run: npm ci | ||||||||||||||||||||||||
| run: bun install --frozen-lockfile | ||||||||||||||||||||||||
|
|
||||||||||||||||||||||||
| - name: Build documentation | ||||||||||||||||||||||||
| run: npm run build | ||||||||||||||||||||||||
| run: bun run build | ||||||||||||||||||||||||
|
|
||||||||||||||||||||||||
| - name: Test that built files exist | ||||||||||||||||||||||||
| run: | | ||||||||||||||||||||||||
|
|
@@ -54,7 +52,7 @@ jobs: | |||||||||||||||||||||||
| # Optional: You could also add a step to test the built site | ||||||||||||||||||||||||
| - name: Test serve (optional) | ||||||||||||||||||||||||
| run: | | ||||||||||||||||||||||||
| npm run serve & | ||||||||||||||||||||||||
| bun run serve & | ||||||||||||||||||||||||
| sleep 5 | ||||||||||||||||||||||||
| curl -f http://localhost:3000 || exit 1 | ||||||||||||||||||||||||
| echo "✅ Documentation serves successfully" | ||||||||||||||||||||||||
| echo "✅ Documentation serves successfully" | ||||||||||||||||||||||||
| Original file line number | Diff line number | Diff line change | ||||||||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
|
|
@@ -22,20 +22,16 @@ jobs: | |||||||||||||||||||||
| ref: ${{ github.event.pull_request.head.ref }} | ||||||||||||||||||||||
| repository: ${{ github.event.pull_request.head.repo.full_name }} | ||||||||||||||||||||||
|
|
||||||||||||||||||||||
| - name: Enable corepack | ||||||||||||||||||||||
| run: corepack enable | ||||||||||||||||||||||
|
|
||||||||||||||||||||||
| - name: Setup Node | ||||||||||||||||||||||
| uses: actions/setup-node@v4 | ||||||||||||||||||||||
| - name: Set up Bun | ||||||||||||||||||||||
| uses: oven-sh/setup-bun@v2 | ||||||||||||||||||||||
| with: | ||||||||||||||||||||||
| node-version: 20 | ||||||||||||||||||||||
| cache: 'pnpm' | ||||||||||||||||||||||
| bun-version: '1.3.5' | ||||||||||||||||||||||
|
Comment on lines
+25
to
+28
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Inconsistent This workflow uses Proposed fix - name: Set up Bun
- uses: oven-sh/setup-bun@v1
+ uses: oven-sh/setup-bun@v2
with:
bun-version: '1.3.5'📝 Committable suggestion
Suggested change
🤖 Prompt for AI Agents |
||||||||||||||||||||||
|
|
||||||||||||||||||||||
| - name: Install deps | ||||||||||||||||||||||
| run: pnpm install --frozen-lockfile | ||||||||||||||||||||||
| run: bun install --frozen-lockfile | ||||||||||||||||||||||
|
|
||||||||||||||||||||||
| - name: Run sync script | ||||||||||||||||||||||
| run: pnpm -w tsx packages/cli/scripts/sync-cli-readme.ts | ||||||||||||||||||||||
| run: bun packages/cli/scripts/sync-cli-readme.ts | ||||||||||||||||||||||
|
|
||||||||||||||||||||||
| - name: Commit changes (same-repo PRs) | ||||||||||||||||||||||
| if: ${{ github.event.pull_request.head.repo.full_name == github.repository }} | ||||||||||||||||||||||
|
|
@@ -55,9 +51,8 @@ jobs: | |||||||||||||||||||||
| run: | | ||||||||||||||||||||||
| if ! git diff --quiet --exit-code packages/cli/README.md; then | ||||||||||||||||||||||
| echo "CLI README is out of sync with root README." >&2 | ||||||||||||||||||||||
| echo "Please run: pnpm -w tsx packages/cli/scripts/sync-cli-readme.ts and commit the updated packages/cli/README.md" >&2 | ||||||||||||||||||||||
| echo "Please run: bun packages/cli/scripts/sync-cli-readme.ts and commit the updated packages/cli/README.md" >&2 | ||||||||||||||||||||||
| exit 1 | ||||||||||||||||||||||
| else | ||||||||||||||||||||||
| echo "CLI README already in sync." | ||||||||||||||||||||||
| fi | ||||||||||||||||||||||
|
|
||||||||||||||||||||||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -14,6 +14,7 @@ build/ | |
| .turbo/ | ||
| .vercel/ | ||
| dist-electron/ | ||
| packages/dexto-*/bin/ | ||
|
|
||
| # Environment files | ||
| .env | ||
|
|
||
Uh oh!
There was an error while loading. Please reload this page.