Skip to content
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(turbo-utils): Remove tsup build #6301

Merged
merged 12 commits into from Nov 2, 2023
Merged

chore(turbo-utils): Remove tsup build #6301

merged 12 commits into from Nov 2, 2023

Conversation

mehulkar
Copy link
Contributor

@mehulkar mehulkar commented Oct 27, 2023

This package does not need to be bundled since it is used only internally in this monorepo. Since we are not publishing it, it does not matter whether it is bundled or not.

package file before after
create-turbo CJS dist/cli.js 1.21 MB 549.34 KB
turbo-ignore CJS dist/cli.js 571.73 KB 455.74 KB
eslint-plugin-turbo CJS dist/index.js 540.35 KB 424.31 KB
@turbo/gen CJS dist/cli.js 556.05 KB 460.46 KB
@turbo/codemod CJS dist/cli.js 1.15 MB 379.58 KB
@turbo/workspaces CJS dist/index.js 523.75 KB 131.29 KB
@turbo/workspaces CJS dist/cli.js 528.08 KB 126.35 KB
@turbo/workspaces ESM dist/index.mjs 401.00 B 4.77 KB
@turbo/workspaces ESM dist/cli.mjs 4.75 KB 412.00 B

Closes TURBO-1542

This package does not need to be bundled since it is used only internally
in this monorepo. Since we are not publishing it, it does not matter whether
it is bundled or not.
@vercel
Copy link

vercel bot commented Oct 27, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
examples-kitchensink-blog 🔄 Building (Inspect) Visit Preview 💬 Add feedback Nov 2, 2023 5:39pm
turbo-site ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 2, 2023 5:39pm
9 Ignored Deployments
Name Status Preview Comments Updated (UTC)
examples-basic-web ⬜️ Ignored (Inspect) Visit Preview Nov 2, 2023 5:39pm
examples-cra-web ⬜️ Ignored (Inspect) Visit Preview Nov 2, 2023 5:39pm
examples-designsystem-docs ⬜️ Ignored (Inspect) Visit Preview Nov 2, 2023 5:39pm
examples-gatsby-web ⬜️ Ignored (Inspect) Visit Preview Nov 2, 2023 5:39pm
examples-native-web ⬜️ Ignored (Inspect) Visit Preview Nov 2, 2023 5:39pm
examples-nonmonorepo ⬜️ Ignored (Inspect) Visit Preview Nov 2, 2023 5:39pm
examples-svelte-web ⬜️ Ignored (Inspect) Visit Preview Nov 2, 2023 5:39pm
examples-tailwind-web ⬜️ Ignored (Inspect) Visit Preview Nov 2, 2023 5:39pm
examples-vite-web ⬜️ Ignored (Inspect) Visit Preview Nov 2, 2023 5:39pm

@github-actions
Copy link
Contributor

✅ This change can build next-swc

@github-actions
Copy link
Contributor

Linux Benchmark for d6d62c6

Test Base PR % Significant %
bench_hmr_to_eval/Turbopack CSR/1000 modules N/A 66.48ms ± 4.76ms N/A N/A
Click to view full benchmark
Test Base PR % Significant %
bench_hmr_to_commit/Turbopack CSR/1000 modules 67.32ms ± 5.49ms 67.22ms ± 5.17ms -0.15%
bench_hmr_to_eval/Turbopack CSR/1000 modules N/A 66.48ms ± 4.76ms N/A N/A
bench_startup/Turbopack CSR/1000 modules 1071.55ms ± 4.28ms 1067.51ms ± 3.83ms -0.38%

@github-actions
Copy link
Contributor

github-actions bot commented Oct 27, 2023

🟢 Turbopack Benchmark CI successful 🟢

Thanks

@github-actions
Copy link
Contributor

github-actions bot commented Oct 27, 2023

⚠️ CI failed ⚠️

The following steps have failed in CI:

  • Turbopack Rust tests (mac/win, non-blocking)

See workflow summary for details

@tknickman
Copy link
Member

Looks good! Let's double check the bundle sizes of packages that are published that depend on this one, but otherwise I'm sold.

@mehulkar

This comment was marked as outdated.

@mehulkar
Copy link
Contributor Author

Looks like I had the dist/ dir sitting around for the utils package and hadn't updated the entrypoints in package.json, so it was the same size. after fixing that, i'm actually seeing a dramatic reduction in size of upstream bundles:

package file before after
turbo-ignore:build: CJS dist/cli.js 571.73 KB 455.74 KB
eslint-plugin-turbo:build: CJS dist/index.js 540.35 KB 424.31 KB
@turbo/workspaces:build: CJS dist/index.js 523.75 KB 131.29 KB
@turbo/workspaces:build: CJS dist/cli.js 528.08 KB 126.35 KB
@turbo/workspaces:build: ESM dist/index.mjs 401.00 B 4.77 KB
@turbo/workspaces:build: ESM dist/cli.mjs 4.75 KB 412.00 B
create-turbo:build: CJS dist/cli.js 1.21 MB 549.34 KB
@turbo/gen:build: CJS dist/cli.js 556.05 KB 460.46 KB
@turbo/codemod:build: CJS dist/cli.js 1.15 MB 379.58 KB

@github-actions
Copy link
Contributor

Linux Benchmark for 30d5c4f

Click to view benchmark
Test Base PR % Significant %
bench_hmr_to_commit/Turbopack CSR/1000 modules 72.24ms ± 5.15ms 74.86ms ± 5.23ms +3.62%
bench_hmr_to_eval/Turbopack CSR/1000 modules 75.19ms ± 5.09ms 72.40ms ± 5.24ms -3.71%
bench_startup/Turbopack CSR/1000 modules 1085.05ms ± 4.22ms 1079.40ms ± 7.72ms -0.52%

Copy link
Contributor

github-actions bot commented Nov 1, 2023

Linux Benchmark for 142cdce

Click to view benchmark
Test Base PR % Significant %
bench_hmr_to_commit/Turbopack CSR/1000 modules 68.86ms ± 4.98ms 69.12ms ± 5.05ms +0.38%
bench_hmr_to_eval/Turbopack CSR/1000 modules 68.45ms ± 4.93ms 67.19ms ± 5.26ms -1.83%
bench_startup/Turbopack CSR/1000 modules 1070.80ms ± 1.54ms 1061.52ms ± 6.38ms -0.87%

Copy link
Contributor

github-actions bot commented Nov 1, 2023

Linux Benchmark for ea44e5b

Click to view benchmark
Test Base PR % Significant %
bench_hmr_to_commit/Turbopack CSR/1000 modules 66.64ms ± 4.71ms 66.52ms ± 5.10ms -0.19%
bench_hmr_to_eval/Turbopack CSR/1000 modules 64.69ms ± 4.97ms 66.70ms ± 4.52ms +3.10%
bench_startup/Turbopack CSR/1000 modules 1062.63ms ± 0.98ms 1066.68ms ± 8.06ms +0.38%

Copy link
Contributor

github-actions bot commented Nov 2, 2023

Linux Benchmark for 0b53bea

Click to view benchmark
Test Base PR % Significant %
bench_hmr_to_commit/Turbopack CSR/1000 modules 71.24ms ± 4.83ms 71.22ms ± 5.25ms -0.03%
bench_hmr_to_eval/Turbopack CSR/1000 modules 68.91ms ± 5.46ms 69.10ms ± 4.88ms +0.27%
bench_startup/Turbopack CSR/1000 modules 1067.23ms ± 2.34ms 1081.18ms ± 7.05ms +1.31%

Copy link
Contributor

github-actions bot commented Nov 2, 2023

Linux Benchmark for 29449cd

Test Base PR % Significant %
bench_startup/Turbopack CSR/1000 modules 1054.01ms ± 2.80ms 1069.04ms ± 2.59ms +1.43% +0.40%
Click to view full benchmark
Test Base PR % Significant %
bench_hmr_to_commit/Turbopack CSR/1000 modules 70.04ms ± 4.60ms 69.63ms ± 5.07ms -0.57%
bench_hmr_to_eval/Turbopack CSR/1000 modules 69.07ms ± 5.07ms 69.69ms ± 5.29ms +0.89%
bench_startup/Turbopack CSR/1000 modules 1054.01ms ± 2.80ms 1069.04ms ± 2.59ms +1.43% +0.40%

Copy link
Contributor

github-actions bot commented Nov 2, 2023

Linux Benchmark for c025b06

Click to view benchmark
Test Base PR % Significant %
bench_hmr_to_commit/Turbopack CSR/1000 modules 67.42ms ± 4.52ms 66.19ms ± 4.90ms -1.83%
bench_hmr_to_eval/Turbopack CSR/1000 modules 64.52ms ± 4.54ms 63.61ms ± 4.51ms -1.40%
bench_startup/Turbopack CSR/1000 modules 1061.34ms ± 3.05ms 1051.95ms ± 8.65ms -0.88%

@mehulkar mehulkar merged commit 894f932 into main Nov 2, 2023
61 of 64 checks passed
@mehulkar mehulkar deleted the mk/rm-tsup-from-utils branch November 2, 2023 19:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants