-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
Export __N_SSG and __N_SSP from the error component #3574
Export __N_SSG and __N_SSP from the error component #3574
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎ 10 Ignored Deployments
|
Benchmark for ed19fb3
Click to view full benchmark
|
|
Benchmark for ed19fb3
Click to view full benchmark
|
90e4eb1
to
2dee8b5
Compare
Benchmark for 90c9acf
Click to view full benchmark
|
2dee8b5
to
8ea58f5
Compare
Benchmark for eb73576Click to view benchmark
|
@@ -2,4 +2,5 @@ module.exports = { | |||
experimental: { | |||
appDir: true, | |||
}, | |||
transpilePackages: ["@turbo/pack-test-harness"], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Any idea why I need to include this for this test, but it's not necessary for the 404 test?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🤷
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This was because these two cases have a tsconfig.json
that overrides the top-level tsconfig.json
, which includes the test harness in its paths list (hence it's compiled without needing transpilePackages).
@@ -2,4 +2,5 @@ module.exports = { | |||
experimental: { | |||
appDir: true, | |||
}, | |||
transpilePackages: ["@turbo/pack-test-harness"], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🤷
925ee28
to
04e445b
Compare
Failing next.js integration test suitesCommit: c751091 Test summary
✅ Fixed tests:
❌ Newly failed tests:
Full test report
Expand output● ReactRefresh › can edit a component without losing state
● ReactRefresh › cyclic dependencies
Expand output● i18n-ignore-rewrite-source-locale with basepath › get public file by skipping locale in rewrite, locale:
● i18n-ignore-rewrite-source-locale with basepath › get public file by skipping locale in rewrite, locale: /en
● i18n-ignore-rewrite-source-locale with basepath › get public file by skipping locale in rewrite, locale: /sv
● i18n-ignore-rewrite-source-locale with basepath › get public file by skipping locale in rewrite, locale: /nl
● i18n-ignore-rewrite-source-locale with basepath › call api by skipping locale in rewrite, locale:
● i18n-ignore-rewrite-source-locale with basepath › call api by skipping locale in rewrite, locale: /en
● i18n-ignore-rewrite-source-locale with basepath › call api by skipping locale in rewrite, locale: /sv
● i18n-ignore-rewrite-source-locale with basepath › call api by skipping locale in rewrite, locale: /nl
● i18n-ignore-rewrite-source-locale with basepath › get _next/static/ files by skipping locale in rewrite, locale:
● i18n-ignore-rewrite-source-locale with basepath › get _next/static/ files by skipping locale in rewrite, locale: /en
● i18n-ignore-rewrite-source-locale with basepath › get _next/static/ files by skipping locale in rewrite, locale: /sv
● i18n-ignore-rewrite-source-locale with basepath › get _next/static/ files by skipping locale in rewrite, locale: /nl
Expand output● Catches Missing getStaticProps › should catch it in dev mode
Expand output● ReactRefreshLogBox turbo › empty _app shows logbox
● ReactRefreshLogBox turbo › empty _document shows logbox
● ReactRefreshLogBox turbo › _app syntax error shows logbox
● ReactRefreshLogBox turbo › _document syntax error shows logbox
Expand output● app-dir alias handling › should handle typescript paths alias correctly
● app-dir alias handling › should resolve css imports from outside with src folder presented
Expand output● bigint API route support › dev mode › should return 200
● bigint API route support › dev mode › should return the BigInt result text
Expand output● ReactRefresh › can edit a component without losing state
● ReactRefresh › cyclic dependencies
Expand output● i18n-ignore-rewrite-source-locale with basepath › get public file by skipping locale in rewrite, locale:
● i18n-ignore-rewrite-source-locale with basepath › get public file by skipping locale in rewrite, locale: /en
● i18n-ignore-rewrite-source-locale with basepath › get public file by skipping locale in rewrite, locale: /sv
● i18n-ignore-rewrite-source-locale with basepath › get public file by skipping locale in rewrite, locale: /nl
● i18n-ignore-rewrite-source-locale with basepath › call api by skipping locale in rewrite, locale:
● i18n-ignore-rewrite-source-locale with basepath › call api by skipping locale in rewrite, locale: /en
● i18n-ignore-rewrite-source-locale with basepath › call api by skipping locale in rewrite, locale: /sv
● i18n-ignore-rewrite-source-locale with basepath › call api by skipping locale in rewrite, locale: /nl
● i18n-ignore-rewrite-source-locale with basepath › get _next/static/ files by skipping locale in rewrite, locale:
● i18n-ignore-rewrite-source-locale with basepath › get _next/static/ files by skipping locale in rewrite, locale: /en
● i18n-ignore-rewrite-source-locale with basepath › get _next/static/ files by skipping locale in rewrite, locale: /sv
● i18n-ignore-rewrite-source-locale with basepath › get _next/static/ files by skipping locale in rewrite, locale: /nl
Expand output● Catches Missing getStaticProps › should catch it in dev mode
Expand output● ReactRefreshLogBox turbo › empty _app shows logbox
● ReactRefreshLogBox turbo › empty _document shows logbox
● ReactRefreshLogBox turbo › _app syntax error shows logbox
● ReactRefreshLogBox turbo › _document syntax error shows logbox
Expand output● app-dir alias handling › should handle typescript paths alias correctly
● app-dir alias handling › should resolve css imports from outside with src folder presented
Expand output● bigint API route support › dev mode › should return 200
● bigint API route support › dev mode › should return the BigInt result text
|
Benchmark for c751091Click to view benchmark
|
Benchmark for d09ee7d
Click to view full benchmark
|
…49736102+kodiakhq[bot]@users.noreply.github.com> # New Features - vercel/turborepo#3540 - vercel/turborepo#3549 - vercel/turborepo#3465 - vercel/turborepo#3550 - vercel/turborepo#3495 - vercel/turborepo#3624 - vercel/turborepo#3600 - vercel/turborepo#3676 - vercel/turborepo#3689 # Fixes - vercel/turborepo#3437 - vercel/turborepo#3542 - vercel/turborepo#3531 - vercel/turborepo#3552 - vercel/turborepo#3551 - vercel/turborepo#3597 - vercel/turborepo#3644 - vercel/turborepo#3623 - vercel/turborepo#3634 - vercel/turborepo#3574 - vercel/turborepo#3673 - vercel/turborepo#3675 - vercel/turborepo#3723 - vercel/turborepo#3677 - vercel/turborepo#3717 - vercel/turborepo#3701 # Performance Improvements - vercel/turborepo#3361 - vercel/turborepo#3619 --------- Co-authored-by: kodiakhq[bot] <49736102+kodiakhq[bot]@users.noreply.github.com>
…#3574) `export *` would be preferable once supported.
…#3574) `export *` would be preferable once supported.
…#3574) `export *` would be preferable once supported.
…#3574) `export *` would be preferable once supported.
…#3574) `export *` would be preferable once supported.
export *
would be preferable once supported.