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

Remove next-dev references and benchmarks #6026

Merged
merged 2 commits into from
Sep 26, 2023
Merged

Conversation

wbinnssmith
Copy link
Member

@wbinnssmith wbinnssmith commented Sep 25, 2023

We've rearchitected Next.js+Turbopack so Turbopack does not run reimplement pieces of Next.js in its devserver. This:

Test Plan: CI

Closes WEB-1653

We've rearchitected Next.js+Turbopack so Turbopack does not run reimplement pieces of Next.js in its devserver. This:

- Removes references and tests of the `next-dev` binary, which is no longer reachable through `next --turbo`. Uses `next-swc` in its place when testing.
- Removes its benchmark suite, which should be covered by `turbopack-bench` by https://github.com/vercel/turbo/tree/main/crates/turbopack-bench

Test Plan: CI
@wbinnssmith wbinnssmith requested a review from a team as a code owner September 25, 2023 21:53
@vercel
Copy link

vercel bot commented Sep 25, 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 Sep 25, 2023 10:42pm
examples-kitchensink-blog 🔄 Building (Inspect) Visit Preview 💬 Add feedback Sep 25, 2023 10:42pm
examples-nonmonorepo 🔄 Building (Inspect) Visit Preview 💬 Add feedback Sep 25, 2023 10:42pm
8 Ignored Deployments
Name Status Preview Comments Updated (UTC)
examples-basic-web ⬜️ Ignored (Inspect) Visit Preview Sep 25, 2023 10:42pm
examples-designsystem-docs ⬜️ Ignored (Inspect) Visit Preview Sep 25, 2023 10:42pm
examples-gatsby-web ⬜️ Ignored (Inspect) Visit Preview Sep 25, 2023 10:42pm
examples-native-web ⬜️ Ignored (Inspect) Visit Preview Sep 25, 2023 10:42pm
examples-svelte-web ⬜️ Ignored (Inspect) Visit Preview Sep 25, 2023 10:42pm
examples-tailwind-web ⬜️ Ignored (Inspect) Visit Preview Sep 25, 2023 10:42pm
examples-vite-web ⬜️ Ignored (Inspect) Visit Preview Sep 25, 2023 10:42pm
turbo-site ⬜️ Ignored (Inspect) Visit Preview Sep 25, 2023 10:42pm

Copy link

@orca-security-us orca-security-us bot left a comment

Choose a reason for hiding this comment

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

Orca Security Scan Summary

Status Check Issues by priority
Passed Passed Secrets high 0   medium 0   low 0   info 0 View in Orca

@github-actions
Copy link
Contributor

✅ This change can build next-swc

@github-actions
Copy link
Contributor

Linux Benchmark for 59bacb3

Click to view benchmark
Test Base PR % Significant %
bench_hmr_to_commit/Turbopack CSR/1000 modules 5349.57µs ± 23.25µs 5316.72µs ± 33.38µs -0.61%
bench_hmr_to_eval/Turbopack CSR/1000 modules 5002.20µs ± 20.91µs 5281.88µs ± 354.97µs +5.59%
bench_startup/Turbopack CSR/1000 modules 982.37ms ± 0.80ms 992.29ms ± 5.47ms +1.01%

@github-actions
Copy link
Contributor

github-actions bot commented Sep 25, 2023

⚠️ CI failed ⚠️

The following steps have failed in CI:

  • Turbopack Rust benchmarks (non-blocking)

See workflow summary for details

@github-actions
Copy link
Contributor

Linux Benchmark for 9b9faec

Click to view benchmark
Test Base PR % Significant %
bench_hmr_to_commit/Turbopack CSR/1000 modules 5309.86µs ± 33.49µs 5318.87µs ± 25.11µs +0.17%
bench_hmr_to_eval/Turbopack CSR/1000 modules 4988.46µs ± 17.61µs 5271.17µs ± 329.34µs +5.67%
bench_startup/Turbopack CSR/1000 modules 976.11ms ± 1.52ms 978.11ms ± 3.35ms +0.21%

@wbinnssmith wbinnssmith marked this pull request as ready for review September 25, 2023 23:08
@github-actions
Copy link
Contributor

MacOS Benchmark for 9b9faec

Test Base PR % Significant %
bench_hmr_to_eval/Turbopack CSR/1000 modules 26.34ms ± 0.14ms 23.66ms ± 0.96ms -10.16% -1.85%
bench_startup/Turbopack CSR/1000 modules 9916.99ms ± 1747.47ms 4293.62ms ± 120.17ms -56.70% -29.40%
Click to view full benchmark
Test Base PR % Significant %
bench_hmr_to_commit/Turbopack CSR/1000 modules 27.38ms ± 0.17ms 27.33ms ± 0.13ms -0.19%
bench_hmr_to_eval/Turbopack CSR/1000 modules 26.34ms ± 0.14ms 23.66ms ± 0.96ms -10.16% -1.85%
bench_startup/Turbopack CSR/1000 modules 9916.99ms ± 1747.47ms 4293.62ms ± 120.17ms -56.70% -29.40%

@github-actions
Copy link
Contributor

Windows Benchmark for 9b9faec

Test Base PR % Significant %
bench_hmr_to_eval/Turbopack CSR/1000 modules 15.50ms ± 0.01ms 15.54ms ± 0.01ms +0.31% +0.06%
Click to view full benchmark
Test Base PR % Significant %
bench_hmr_to_commit/Turbopack CSR/1000 modules 15.54ms ± 0.01ms 15.54ms ± 0.01ms +0.02%
bench_hmr_to_eval/Turbopack CSR/1000 modules 15.50ms ± 0.01ms 15.54ms ± 0.01ms +0.31% +0.06%
bench_startup/Turbopack CSR/1000 modules 3434.63ms ± 3.83ms 3461.94ms ± 41.39ms +0.80%

@sokra sokra merged commit e4395ef into main Sep 26, 2023
63 of 69 checks passed
@sokra sokra deleted the wbinnssmith/remove-next-dev branch September 26, 2023 04:26
Zertsov pushed a commit that referenced this pull request Sep 27, 2023
We've rearchitected Next.js+Turbopack so Turbopack does not run
reimplement pieces of Next.js in its devserver. This:

- Removes references and tests of the `next-dev` binary, which is no
longer reachable through `next --turbo`. Uses `next-swc` in its place
when testing.
- Removes its benchmark suite, which should be covered by
`turbopack-bench` by
https://github.com/vercel/turbo/tree/main/crates/turbopack-bench

Test Plan: CI


Closes WEB-1653
sokra added a commit to vercel/next.js that referenced this pull request Sep 28, 2023
### What?

see vercel/turborepo#5992

### Turobopack changes

* vercel/turborepo#6009 <!-- OJ Kwon - ci(workflow):
update test filter -->
* vercel/turborepo#6026 <!-- Will Binns-Smith -
Remove next-dev references and benchmarks -->
* vercel/turborepo#6038 <!-- Tim Neutkens - Remove
test-prod action -->
* vercel/turborepo#6039 <!-- Tim Neutkens - Fix
action dependency -->
* ~vercel/turborepo#6036 <!-- Will Binns-Smith -
Turbopack: add support for an asset prefix (and basePath in Next.js) -->
* vercel/turborepo#5992 <!-- Tobias Koppers -
refactor TaskScopes to use an aggregation tree -->


Closes WEB-1622
ForsakenHarmony pushed a commit to vercel/next.js that referenced this pull request Jul 25, 2024
We've rearchitected Next.js+Turbopack so Turbopack does not run
reimplement pieces of Next.js in its devserver. This:

- Removes references and tests of the `next-dev` binary, which is no
longer reachable through `next --turbo`. Uses `next-swc` in its place
when testing.
- Removes its benchmark suite, which should be covered by
`turbopack-bench` by
https://github.com/vercel/turbo/tree/main/crates/turbopack-bench

Test Plan: CI


Closes WEB-1653
ForsakenHarmony pushed a commit to vercel/next.js that referenced this pull request Jul 29, 2024
We've rearchitected Next.js+Turbopack so Turbopack does not run
reimplement pieces of Next.js in its devserver. This:

- Removes references and tests of the `next-dev` binary, which is no
longer reachable through `next --turbo`. Uses `next-swc` in its place
when testing.
- Removes its benchmark suite, which should be covered by
`turbopack-bench` by
https://github.com/vercel/turbo/tree/main/crates/turbopack-bench

Test Plan: CI


Closes WEB-1653
ForsakenHarmony pushed a commit to vercel/next.js that referenced this pull request Aug 1, 2024
We've rearchitected Next.js+Turbopack so Turbopack does not run
reimplement pieces of Next.js in its devserver. This:

- Removes references and tests of the `next-dev` binary, which is no
longer reachable through `next --turbo`. Uses `next-swc` in its place
when testing.
- Removes its benchmark suite, which should be covered by
`turbopack-bench` by
https://github.com/vercel/turbo/tree/main/crates/turbopack-bench

Test Plan: CI


Closes WEB-1653
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.

2 participants