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

Transformation code necessary to support Server Actions #5705

Merged
merged 6 commits into from
Aug 16, 2023

Conversation

jridgewell
Copy link
Contributor

@jridgewell jridgewell commented Aug 11, 2023

Description

This adds the comments that are added during transforms to out file outputs.

Testing Instructions

cargo nextest run -E 'test(snapshot)'

Closes WEB-1388

@vercel
Copy link

vercel bot commented Aug 11, 2023

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

Name Status Preview Comments Updated (UTC)
examples-cra-web 🔄 Building (Inspect) Visit Preview 💬 Add feedback Aug 16, 2023 0:23am
examples-nonmonorepo 🔄 Building (Inspect) Visit Preview 💬 Add feedback Aug 16, 2023 0:23am
turbo-site ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 16, 2023 0:23am
8 Ignored Deployments
Name Status Preview Comments Updated (UTC)
examples-basic-web ⬜️ Ignored (Inspect) Visit Preview Aug 16, 2023 0:23am
examples-designsystem-docs ⬜️ Ignored (Inspect) Visit Preview Aug 16, 2023 0:23am
examples-gatsby-web ⬜️ Ignored (Inspect) Visit Preview Aug 16, 2023 0:23am
examples-kitchensink-blog ⬜️ Ignored (Inspect) Visit Preview Aug 16, 2023 0:23am
examples-native-web ⬜️ Ignored (Inspect) Visit Preview Aug 16, 2023 0:23am
examples-svelte-web ⬜️ Ignored (Inspect) Visit Preview Aug 16, 2023 0:23am
examples-tailwind-web ⬜️ Ignored (Inspect) Visit Preview Aug 16, 2023 0:23am
examples-vite-web ⬜️ Ignored (Inspect) Visit Preview Aug 16, 2023 0:23am

@github-actions
Copy link
Contributor

github-actions bot commented Aug 11, 2023

✅ This change can build next-swc

@github-actions
Copy link
Contributor

github-actions bot commented Aug 11, 2023

⚠️ CI failed ⚠️

The following steps have failed in CI:

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

See workflow summary for details

@github-actions
Copy link
Contributor

Linux Benchmark for c411ba6

Test Base PR % Significant %
bench_startup/Turbopack CSR/1000 modules 989.63ms ± 1.26ms 1000.74ms ± 2.93ms +1.12% +0.28%
Click to view full benchmark
Test Base PR % Significant %
bench_hmr_to_commit/Turbopack CSR/1000 modules 5417.83µs ± 22.53µs 5367.15µs ± 29.76µs -0.94%
bench_hmr_to_eval/Turbopack CSR/1000 modules 5066.53µs ± 10.66µs 5055.48µs ± 43.44µs -0.22%
bench_startup/Turbopack CSR/1000 modules 989.63ms ± 1.26ms 1000.74ms ± 2.93ms +1.12% +0.28%

@github-actions
Copy link
Contributor

MacOS Benchmark for c411ba6

Test Base PR % Significant %
bench_startup/Turbopack CSR/1000 modules 14.65s ± 2.09s 3739.62ms ± 158.75ms -74.47% -61.28%
Click to view full benchmark
Test Base PR % Significant %
bench_hmr_to_commit/Turbopack CSR/1000 modules 27.34ms ± 0.17ms 26.74ms ± 0.25ms -2.21%
bench_hmr_to_eval/Turbopack CSR/1000 modules 26.49ms ± 0.17ms 26.03ms ± 0.15ms -1.75%
bench_startup/Turbopack CSR/1000 modules 14.65s ± 2.09s 3739.62ms ± 158.75ms -74.47% -61.28%

@github-actions
Copy link
Contributor

Linux Benchmark for 82c2e3a

Click to view benchmark
Test Base PR % Significant %
bench_hmr_to_commit/Turbopack CSR/1000 modules 7042.15µs ± 21.26µs 7046.07µs ± 23.75µs +0.06%
bench_hmr_to_eval/Turbopack CSR/1000 modules 6400.93µs ± 26.78µs 6753.58µs ± 261.46µs +5.51%
bench_startup/Turbopack CSR/1000 modules 1044.52ms ± 2.41ms 1060.29ms ± 9.39ms +1.51%

@github-actions
Copy link
Contributor

MacOS Benchmark for 82c2e3a

Test Base PR % Significant %
bench_hmr_to_commit/Turbopack CSR/1000 modules 27.26ms ± 0.07ms 26.85ms ± 0.11ms -1.50% -0.18%
Click to view full benchmark
Test Base PR % Significant %
bench_hmr_to_commit/Turbopack CSR/1000 modules 27.26ms ± 0.07ms 26.85ms ± 0.11ms -1.50% -0.18%
bench_hmr_to_eval/Turbopack CSR/1000 modules 26.89ms ± 0.37ms 27.47ms ± 0.21ms +2.18%
bench_startup/Turbopack CSR/1000 modules 3189.95ms ± 19.94ms 3360.87ms ± 88.86ms +5.36%

@github-actions
Copy link
Contributor

Linux Benchmark for b7a3d26

Click to view benchmark
Test Base PR % Significant %
bench_hmr_to_commit/Turbopack CSR/1000 modules 5440.96µs ± 12.27µs 5412.31µs ± 40.45µs -0.53%
bench_hmr_to_eval/Turbopack CSR/1000 modules 5050.90µs ± 31.72µs 5153.56µs ± 114.36µs +2.03%
bench_startup/Turbopack CSR/1000 modules 996.79ms ± 0.69ms 1003.99ms ± 5.33ms +0.72%

@github-actions
Copy link
Contributor

MacOS Benchmark for b7a3d26

Test Base PR % Significant %
bench_hmr_to_eval/Turbopack CSR/1000 modules 22.02ms ± 0.83ms 25.97ms ± 0.22ms +17.93% +7.85%
bench_startup/Turbopack CSR/1000 modules 10.26s ± 1.94s 3606.88ms ± 91.05ms -64.85% -40.71%
Click to view full benchmark
Test Base PR % Significant %
bench_hmr_to_commit/Turbopack CSR/1000 modules 27.01ms ± 0.03ms 26.91ms ± 0.20ms -0.36%
bench_hmr_to_eval/Turbopack CSR/1000 modules 22.02ms ± 0.83ms 25.97ms ± 0.22ms +17.93% +7.85%
bench_startup/Turbopack CSR/1000 modules 10.26s ± 1.94s 3606.88ms ± 91.05ms -64.85% -40.71%

@github-actions
Copy link
Contributor

Linux Benchmark for 80e23de

Click to view benchmark
Test Base PR % Significant %
bench_hmr_to_commit/Turbopack CSR/1000 modules 5385.88µs ± 16.20µs 5402.50µs ± 15.18µs +0.31%
bench_hmr_to_eval/Turbopack CSR/1000 modules 5004.94µs ± 25.49µs 5028.85µs ± 14.09µs +0.48%
bench_startup/Turbopack CSR/1000 modules 992.31ms ± 0.52ms 994.44ms ± 3.84ms +0.21%

@github-actions
Copy link
Contributor

MacOS Benchmark for 80e23de

Test Base PR % Significant %
bench_hmr_to_eval/Turbopack CSR/1000 modules 23.89ms ± 0.50ms 25.88ms ± 0.26ms +8.34% +1.90%
Click to view full benchmark
Test Base PR % Significant %
bench_hmr_to_commit/Turbopack CSR/1000 modules 25.48ms ± 0.17ms 25.79ms ± 0.09ms +1.19%
bench_hmr_to_eval/Turbopack CSR/1000 modules 23.89ms ± 0.50ms 25.88ms ± 0.26ms +8.34% +1.90%
bench_startup/Turbopack CSR/1000 modules 3241.29ms ± 115.36ms 3173.42ms ± 69.59ms -2.09%

Copy link
Member

@kdy1 kdy1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems like a simple change 😄

@kodiakhq kodiakhq bot merged commit f74ff32 into main Aug 16, 2023
47 of 49 checks passed
@kodiakhq kodiakhq bot deleted the jrl-server-actions branch August 16, 2023 18:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
created-by: turbopack owned-by: turbopack pr: automerge Kodiak will merge these automatically after checks pass
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants