-
Notifications
You must be signed in to change notification settings - Fork 2.1k
ci(workflow): format integration test results #3534
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
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
8 Ignored Deployments
|
79b959d
to
6c64b04
Compare
8419b45
to
7064c2d
Compare
7064c2d
to
3e7f116
Compare
Benchmark for b688e73Click to view benchmark
|
fae6420
to
c2ee236
Compare
Benchmark for 7fdc2f1Click to view benchmark
|
5e59955
to
61092de
Compare
Benchmark for e3b53afClick to view benchmark
|
d947546
to
9b1c43c
Compare
9b1c43c
to
53babbe
Compare
Benchmark for bd97594Click to view benchmark
|
|
53babbe
to
4ae88f2
Compare
Failing next.js integration test suitesCommit: d6b21e2 Test summary
✅ Fixed tests: Full test report
Expand output● Legacy decorators SWC option › should compile with legacy decorators enabled
Expand output● manual-client-base-path › should not update with basePath on mount /
● manual-client-base-path › should not update with basePath on mount /another
● manual-client-base-path › should not update with basePath on mount /dynamic/first
● manual-client-base-path › should not update with basePath on mount /dynamic/second
● manual-client-base-path › should navigate correctly from index
● manual-client-base-path › should navigate correctly from another
Expand output● 404 Page Support with _app › dev mode › should not show pages/404 GIP error if _app has GIP
Expand output● ReactRefresh › can edit a component without losing state
● ReactRefresh › cyclic dependencies
Expand output● app dir rendering › should serve app/page.server.js at /
● app dir rendering › SSR only › should run data in layout and page
● app dir rendering › SSR only › should run data fetch in parallel
● app dir rendering › static only › should run data in layout and page
● app dir rendering › static only › should run data in parallel during development
● app dir rendering › ISR › should revalidate the page when revalidate is configured
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● Legacy decorators SWC option › should compile with legacy decorators enabled
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● 404 Page Support with _app › dev mode › should not show pages/404 GIP error if _app has GIP
Expand output● ReactRefresh › can edit a component without losing state
● ReactRefresh › cyclic dependencies
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
|
4ae88f2
to
560a8cb
Compare
Benchmark for 582b52aClick to view benchmark
|
811d4b7
to
b566539
Compare
Benchmark for ef6f403
Click to view full benchmark
|
b566539
to
b963904
Compare
Benchmark for d6b21e2Click to view benchmark
|
Closes WEB-486. This PR amends stat report action formatting - mainly introducing diff summary between base (`main` for the PR, `release` to `release` for new next.js release). This summary will be piped into slack channel once this PR lands. There maybe some things to amend further, but primary logic seems to working. Please note current base (`main`) results are incorrectly collected, makes diff summary looks weird. We may need wipe out incorrect data.
Requires #3534 Partially progresses WEB-460 This PR creates a plain text summary, so subsequent workflow can download it via download-artifact then broadcast as needed. We'll use it to posting into Slack channel. Action will only generate text file when we supposed to report (currently, release-to-release) so if download-artifact does not contain text-summary.md, it can skip to broadcast into Slack channel.
Closes WEB-486.
This PR amends stat report action formatting - mainly introducing diff summary between base (
main
for the PR,release
torelease
for new next.js release). This summary will be piped into slack channel once this PR lands.There maybe some things to amend further, but primary logic seems to working. Please note current base (
main
) results are incorrectly collected, makes diff summary looks weird. We may need wipe out incorrect data.