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

Upgrade Emotion example to React 17 and Emotion 11 and use automatic JSX runtime #18620

Merged
merged 10 commits into from
Nov 19, 2020
Merged

Conversation

karlhorky
Copy link
Contributor

@karlhorky karlhorky commented Nov 1, 2020

‼️ Blocked on #18096

Probably resolved here: #18096 (comment)

Ref: emotion-js/emotion#2062
Ref: emotion-js/emotion#2041 (comment)

Example upgrade here: upleveled/next-js-example-sep-2020@f46407e

@ijjk ijjk added the examples Issue was opened via the examples template. label Nov 1, 2020
@ijjk
Copy link
Member

ijjk commented Nov 1, 2020

Stats from current PR

Default Server Mode (Increase detected ⚠️)
General
vercel/next.js canary karlhorky/next.js patch-1 Change
buildDuration 12.8s 13s ⚠️ +188ms
nodeModulesSize 90.1 MB 90.1 MB
Page Load Tests Overall increase ✓
vercel/next.js canary karlhorky/next.js patch-1 Change
/ failed reqs 0 0
/ total time (seconds) 2.517 2.448 -0.07
/ avg req/sec 993.22 1021.31 +28.09
/error-in-render failed reqs 0 0
/error-in-render total time (seconds) 1.322 1.309 -0.01
/error-in-render avg req/sec 1890.42 1909.96 +19.54
Client Bundles (main, webpack, commons)
vercel/next.js canary karlhorky/next.js patch-1 Change
677f882d2ed8..3aa9.js gzip 11.3 kB 11.3 kB
framework.HASH.js gzip 39 kB 39 kB
main-a9d3c03..afc6.js gzip 7.35 kB 7.35 kB
webpack-e067..f178.js gzip 751 B 751 B
Overall change 58.3 kB 58.3 kB
Client Bundles (main, webpack, commons) Modern
vercel/next.js canary karlhorky/next.js patch-1 Change
677f882d2ed8..dule.js gzip 7.03 kB 7.03 kB
framework.HA..dule.js gzip 39 kB 39 kB
main-3914a19..dule.js gzip 6.35 kB 6.35 kB
webpack-07c5..dule.js gzip 751 B 751 B
Overall change 53.1 kB 53.1 kB
Legacy Client Bundles (polyfills)
vercel/next.js canary karlhorky/next.js patch-1 Change
polyfills-4b..e242.js gzip 31 kB 31 kB
Overall change 31 kB 31 kB
Client Pages
vercel/next.js canary karlhorky/next.js patch-1 Change
_app-9a0b9e1..b37e.js gzip 1.28 kB 1.28 kB
_error-ed1b0..8fbd.js gzip 3.44 kB 3.44 kB
hooks-89731c..c609.js gzip 887 B 887 B
index-17468f..5d83.js gzip 227 B 227 B
link-e71be6d..3531.js gzip 1.34 kB 1.34 kB
routerDirect..924c.js gzip 284 B 284 B
withRouter-7..c13d.js gzip 284 B 284 B
Overall change 7.75 kB 7.75 kB
Client Pages Modern
vercel/next.js canary karlhorky/next.js patch-1 Change
_app-75d3a82..dule.js gzip 625 B 625 B
_error-4469a..dule.js gzip 2.29 kB 2.29 kB
hooks-cbf13f..dule.js gzip 387 B 387 B
index-b9a643..dule.js gzip 226 B 226 B
link-6d15176..dule.js gzip 1.31 kB 1.31 kB
routerDirect..dule.js gzip 284 B 284 B
withRouter-f..dule.js gzip 282 B 282 B
Overall change 5.41 kB 5.41 kB
Client Build Manifests
vercel/next.js canary karlhorky/next.js patch-1 Change
_buildManifest.js gzip 323 B 323 B
_buildManife..dule.js gzip 329 B 329 B
Overall change 652 B 652 B
Rendered Page Sizes
vercel/next.js canary karlhorky/next.js patch-1 Change
index.html gzip 1 kB 1 kB
link.html gzip 1.01 kB 1.01 kB
withRouter.html gzip 996 B 996 B
Overall change 3.01 kB 3.01 kB

Serverless Mode
General
vercel/next.js canary karlhorky/next.js patch-1 Change
buildDuration 15s 14.9s -10ms
nodeModulesSize 90.1 MB 90.1 MB
Client Bundles (main, webpack, commons)
vercel/next.js canary karlhorky/next.js patch-1 Change
677f882d2ed8..3aa9.js gzip 11.3 kB 11.3 kB
framework.HASH.js gzip 39 kB 39 kB
main-a9d3c03..afc6.js gzip 7.35 kB 7.35 kB
webpack-e067..f178.js gzip 751 B 751 B
Overall change 58.3 kB 58.3 kB
Client Bundles (main, webpack, commons) Modern
vercel/next.js canary karlhorky/next.js patch-1 Change
677f882d2ed8..dule.js gzip 7.03 kB 7.03 kB
framework.HA..dule.js gzip 39 kB 39 kB
main-3914a19..dule.js gzip 6.35 kB 6.35 kB
webpack-07c5..dule.js gzip 751 B 751 B
Overall change 53.1 kB 53.1 kB
Legacy Client Bundles (polyfills)
vercel/next.js canary karlhorky/next.js patch-1 Change
polyfills-4b..e242.js gzip 31 kB 31 kB
Overall change 31 kB 31 kB
Client Pages
vercel/next.js canary karlhorky/next.js patch-1 Change
_app-9a0b9e1..b37e.js gzip 1.28 kB 1.28 kB
_error-ed1b0..8fbd.js gzip 3.44 kB 3.44 kB
hooks-89731c..c609.js gzip 887 B 887 B
index-17468f..5d83.js gzip 227 B 227 B
link-e71be6d..3531.js gzip 1.34 kB 1.34 kB
routerDirect..924c.js gzip 284 B 284 B
withRouter-7..c13d.js gzip 284 B 284 B
Overall change 7.75 kB 7.75 kB
Client Pages Modern
vercel/next.js canary karlhorky/next.js patch-1 Change
_app-75d3a82..dule.js gzip 625 B 625 B
_error-4469a..dule.js gzip 2.29 kB 2.29 kB
hooks-cbf13f..dule.js gzip 387 B 387 B
index-b9a643..dule.js gzip 226 B 226 B
link-6d15176..dule.js gzip 1.31 kB 1.31 kB
routerDirect..dule.js gzip 284 B 284 B
withRouter-f..dule.js gzip 282 B 282 B
Overall change 5.41 kB 5.41 kB
Client Build Manifests
vercel/next.js canary karlhorky/next.js patch-1 Change
_buildManifest.js gzip 323 B 323 B
_buildManife..dule.js gzip 329 B 329 B
Overall change 652 B 652 B
Serverless bundles
vercel/next.js canary karlhorky/next.js patch-1 Change
_error.js 1.06 MB 1.06 MB
404.html 4.73 kB 4.73 kB
hooks.html 3.92 kB 3.92 kB
index.js 1.06 MB 1.06 MB
link.js 1.11 MB 1.11 MB
routerDirect.js 1.1 MB 1.1 MB
withRouter.js 1.1 MB 1.1 MB
Overall change 5.43 MB 5.43 MB
Commit: 8a113bc

@karlhorky
Copy link
Contributor Author

karlhorky commented Nov 1, 2020

Hmm... may want to hold off on this until this _jsxs is not defined error is resolved:

#18096 (comment)

Also in Emotion repo: emotion-js/emotion#2064

Copy link
Member

@lfades lfades left a comment

Choose a reason for hiding this comment

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

@karlhorky 👋 I'll request changes per your comments. Let me know when this is ready to go and I'll review it 🕵️

@karlhorky karlhorky changed the title Upgrade Emotion example to React 17 and use automatic JSX runtime Upgrade Emotion example to React 17 and Emotion 11 and use automatic JSX runtime Nov 18, 2020
@vercel
Copy link

vercel bot commented Nov 18, 2020

@karlhorky is attempting to deploy a commit to the Vercel Team on Vercel.

A member of the Team first needs to authorize it.

@karlhorky
Copy link
Contributor Author

karlhorky commented Nov 18, 2020

Ok, @lfades I have asked #18096 (comment), but I think this is the version of configuration and setup that is currently recommended.

Let's wait to see how I get answered over there, but I think unless there are big surprises this should be good to go.

@ijjk
Copy link
Member

ijjk commented Nov 18, 2020

Stats from current PR

Default Server Mode (Increase detected ⚠️)
General
vercel/next.js canary karlhorky/next.js patch-1 Change
buildDuration 14.9s 14.9s ⚠️ +44ms
nodeModulesSize 85 MB 85 MB
Page Load Tests Overall increase ✓
vercel/next.js canary karlhorky/next.js patch-1 Change
/ failed reqs 0 0
/ total time (seconds) 2.767 2.684 -0.08
/ avg req/sec 903.65 931.42 +27.77
/error-in-render failed reqs 0 0
/error-in-render total time (seconds) 1.637 1.657 ⚠️ +0.02
/error-in-render avg req/sec 1527.13 1509.17 ⚠️ -17.96
Client Bundles (main, webpack, commons)
vercel/next.js canary karlhorky/next.js patch-1 Change
677f882d2ed8..776c.js gzip 12.7 kB 12.7 kB
framework.HASH.js gzip 39 kB 39 kB
main-f1a49fb..e45e.js gzip 6.52 kB 6.52 kB
webpack-e067..f178.js gzip 751 B 751 B
Overall change 58.9 kB 58.9 kB
Client Bundles (main, webpack, commons) Modern
vercel/next.js canary karlhorky/next.js patch-1 Change
677f882d2ed8..dule.js gzip 8.26 kB 8.26 kB
framework.HA..dule.js gzip 39 kB 39 kB
main-28351af..dule.js gzip 5.63 kB 5.63 kB
webpack-07c5..dule.js gzip 751 B 751 B
Overall change 53.6 kB 53.6 kB
Legacy Client Bundles (polyfills)
vercel/next.js canary karlhorky/next.js patch-1 Change
polyfills-4b..e242.js gzip 31 kB 31 kB
Overall change 31 kB 31 kB
Client Pages
vercel/next.js canary karlhorky/next.js patch-1 Change
_app-3b0cf13..85f8.js gzip 1.28 kB 1.28 kB
_error-6f635..c393.js gzip 3.44 kB 3.44 kB
hooks-d4ffc3..9e0f.js gzip 887 B 887 B
index-17468f..5d83.js gzip 227 B 227 B
link-b618194..5477.js gzip 1.61 kB 1.61 kB
routerDirect..924c.js gzip 284 B 284 B
withRouter-7..c13d.js gzip 284 B 284 B
Overall change 8.01 kB 8.01 kB
Client Pages Modern
vercel/next.js canary karlhorky/next.js patch-1 Change
_app-75d3a82..dule.js gzip 625 B 625 B
_error-f2fcc..dule.js gzip 2.29 kB 2.29 kB
hooks-cbf13f..dule.js gzip 387 B 387 B
index-b9a643..dule.js gzip 226 B 226 B
link-9741e7f..dule.js gzip 1.55 kB 1.55 kB
routerDirect..dule.js gzip 284 B 284 B
withRouter-f..dule.js gzip 282 B 282 B
Overall change 5.64 kB 5.64 kB
Client Build Manifests
vercel/next.js canary karlhorky/next.js patch-1 Change
_buildManifest.js gzip 321 B 321 B
_buildManife..dule.js gzip 329 B 329 B
Overall change 650 B 650 B
Rendered Page Sizes
vercel/next.js canary karlhorky/next.js patch-1 Change
index.html gzip 966 B 966 B
link.html gzip 974 B 974 B
withRouter.html gzip 959 B 959 B
Overall change 2.9 kB 2.9 kB

Serverless Mode
General
vercel/next.js canary karlhorky/next.js patch-1 Change
buildDuration 16.9s 17s ⚠️ +120ms
nodeModulesSize 85 MB 85 MB
Client Bundles (main, webpack, commons)
vercel/next.js canary karlhorky/next.js patch-1 Change
677f882d2ed8..776c.js gzip 12.7 kB 12.7 kB
framework.HASH.js gzip 39 kB 39 kB
main-f1a49fb..e45e.js gzip 6.52 kB 6.52 kB
webpack-e067..f178.js gzip 751 B 751 B
Overall change 58.9 kB 58.9 kB
Client Bundles (main, webpack, commons) Modern
vercel/next.js canary karlhorky/next.js patch-1 Change
677f882d2ed8..dule.js gzip 8.26 kB 8.26 kB
framework.HA..dule.js gzip 39 kB 39 kB
main-28351af..dule.js gzip 5.63 kB 5.63 kB
webpack-07c5..dule.js gzip 751 B 751 B
Overall change 53.6 kB 53.6 kB
Legacy Client Bundles (polyfills)
vercel/next.js canary karlhorky/next.js patch-1 Change
polyfills-4b..e242.js gzip 31 kB 31 kB
Overall change 31 kB 31 kB
Client Pages
vercel/next.js canary karlhorky/next.js patch-1 Change
_app-3b0cf13..85f8.js gzip 1.28 kB 1.28 kB
_error-6f635..c393.js gzip 3.44 kB 3.44 kB
hooks-d4ffc3..9e0f.js gzip 887 B 887 B
index-17468f..5d83.js gzip 227 B 227 B
link-b618194..5477.js gzip 1.61 kB 1.61 kB
routerDirect..924c.js gzip 284 B 284 B
withRouter-7..c13d.js gzip 284 B 284 B
Overall change 8.01 kB 8.01 kB
Client Pages Modern
vercel/next.js canary karlhorky/next.js patch-1 Change
_app-75d3a82..dule.js gzip 625 B 625 B
_error-f2fcc..dule.js gzip 2.29 kB 2.29 kB
hooks-cbf13f..dule.js gzip 387 B 387 B
index-b9a643..dule.js gzip 226 B 226 B
link-9741e7f..dule.js gzip 1.55 kB 1.55 kB
routerDirect..dule.js gzip 284 B 284 B
withRouter-f..dule.js gzip 282 B 282 B
Overall change 5.64 kB 5.64 kB
Client Build Manifests
vercel/next.js canary karlhorky/next.js patch-1 Change
_buildManifest.js gzip 321 B 321 B
_buildManife..dule.js gzip 329 B 329 B
Overall change 650 B 650 B
Serverless bundles
vercel/next.js canary karlhorky/next.js patch-1 Change
_error.js 918 kB 918 kB
404.html 4.61 kB 4.61 kB
hooks.html 3.86 kB 3.86 kB
index.js 918 kB 918 kB
link.js 977 kB 977 kB
routerDirect.js 969 kB 969 kB
withRouter.js 969 kB 969 kB
Overall change 4.76 MB 4.76 MB
Commit: 681a441

@ijjk
Copy link
Member

ijjk commented Nov 18, 2020

Stats from current PR

Default Server Mode (Decrease detected ✓)
General
vercel/next.js canary karlhorky/next.js patch-1 Change
buildDuration 12.3s 12.6s ⚠️ +274ms
nodeModulesSize 85 MB 85 MB
Page Load Tests Overall decrease ⚠️
vercel/next.js canary karlhorky/next.js patch-1 Change
/ failed reqs 0 0
/ total time (seconds) 2.229 2.459 ⚠️ +0.23
/ avg req/sec 1121.54 1016.52 ⚠️ -105.02
/error-in-render failed reqs 0 0
/error-in-render total time (seconds) 1.234 1.266 ⚠️ +0.03
/error-in-render avg req/sec 2026.72 1974.88 ⚠️ -51.84
Client Bundles (main, webpack, commons)
vercel/next.js canary karlhorky/next.js patch-1 Change
677f882d2ed8..776c.js gzip 12.7 kB 12.7 kB
framework.HASH.js gzip 39 kB 39 kB
main-f1a49fb..e45e.js gzip 6.52 kB 6.52 kB
webpack-e067..f178.js gzip 751 B 751 B
Overall change 58.9 kB 58.9 kB
Client Bundles (main, webpack, commons) Modern
vercel/next.js canary karlhorky/next.js patch-1 Change
677f882d2ed8..dule.js gzip 8.26 kB 8.26 kB
framework.HA..dule.js gzip 39 kB 39 kB
main-28351af..dule.js gzip 5.63 kB 5.63 kB
webpack-07c5..dule.js gzip 751 B 751 B
Overall change 53.6 kB 53.6 kB
Legacy Client Bundles (polyfills)
vercel/next.js canary karlhorky/next.js patch-1 Change
polyfills-4b..e242.js gzip 31 kB 31 kB
Overall change 31 kB 31 kB
Client Pages
vercel/next.js canary karlhorky/next.js patch-1 Change
_app-3b0cf13..85f8.js gzip 1.28 kB 1.28 kB
_error-6f635..c393.js gzip 3.44 kB 3.44 kB
hooks-d4ffc3..9e0f.js gzip 887 B 887 B
index-17468f..5d83.js gzip 227 B 227 B
link-b618194..5477.js gzip 1.61 kB 1.61 kB
routerDirect..924c.js gzip 284 B 284 B
withRouter-7..c13d.js gzip 284 B 284 B
Overall change 8.01 kB 8.01 kB
Client Pages Modern
vercel/next.js canary karlhorky/next.js patch-1 Change
_app-75d3a82..dule.js gzip 625 B 625 B
_error-f2fcc..dule.js gzip 2.29 kB 2.29 kB
hooks-cbf13f..dule.js gzip 387 B 387 B
index-b9a643..dule.js gzip 226 B 226 B
link-9741e7f..dule.js gzip 1.55 kB 1.55 kB
routerDirect..dule.js gzip 284 B 284 B
withRouter-f..dule.js gzip 282 B 282 B
Overall change 5.64 kB 5.64 kB
Client Build Manifests
vercel/next.js canary karlhorky/next.js patch-1 Change
_buildManifest.js gzip 321 B 321 B
_buildManife..dule.js gzip 329 B 329 B
Overall change 650 B 650 B
Rendered Page Sizes
vercel/next.js canary karlhorky/next.js patch-1 Change
index.html gzip 966 B 966 B
link.html gzip 974 B 974 B
withRouter.html gzip 959 B 959 B
Overall change 2.9 kB 2.9 kB

Serverless Mode
General
vercel/next.js canary karlhorky/next.js patch-1 Change
buildDuration 15s 14.9s -16ms
nodeModulesSize 85 MB 85 MB
Client Bundles (main, webpack, commons)
vercel/next.js canary karlhorky/next.js patch-1 Change
677f882d2ed8..776c.js gzip 12.7 kB 12.7 kB
framework.HASH.js gzip 39 kB 39 kB
main-f1a49fb..e45e.js gzip 6.52 kB 6.52 kB
webpack-e067..f178.js gzip 751 B 751 B
Overall change 58.9 kB 58.9 kB
Client Bundles (main, webpack, commons) Modern
vercel/next.js canary karlhorky/next.js patch-1 Change
677f882d2ed8..dule.js gzip 8.26 kB 8.26 kB
framework.HA..dule.js gzip 39 kB 39 kB
main-28351af..dule.js gzip 5.63 kB 5.63 kB
webpack-07c5..dule.js gzip 751 B 751 B
Overall change 53.6 kB 53.6 kB
Legacy Client Bundles (polyfills)
vercel/next.js canary karlhorky/next.js patch-1 Change
polyfills-4b..e242.js gzip 31 kB 31 kB
Overall change 31 kB 31 kB
Client Pages
vercel/next.js canary karlhorky/next.js patch-1 Change
_app-3b0cf13..85f8.js gzip 1.28 kB 1.28 kB
_error-6f635..c393.js gzip 3.44 kB 3.44 kB
hooks-d4ffc3..9e0f.js gzip 887 B 887 B
index-17468f..5d83.js gzip 227 B 227 B
link-b618194..5477.js gzip 1.61 kB 1.61 kB
routerDirect..924c.js gzip 284 B 284 B
withRouter-7..c13d.js gzip 284 B 284 B
Overall change 8.01 kB 8.01 kB
Client Pages Modern
vercel/next.js canary karlhorky/next.js patch-1 Change
_app-75d3a82..dule.js gzip 625 B 625 B
_error-f2fcc..dule.js gzip 2.29 kB 2.29 kB
hooks-cbf13f..dule.js gzip 387 B 387 B
index-b9a643..dule.js gzip 226 B 226 B
link-9741e7f..dule.js gzip 1.55 kB 1.55 kB
routerDirect..dule.js gzip 284 B 284 B
withRouter-f..dule.js gzip 282 B 282 B
Overall change 5.64 kB 5.64 kB
Client Build Manifests
vercel/next.js canary karlhorky/next.js patch-1 Change
_buildManifest.js gzip 321 B 321 B
_buildManife..dule.js gzip 329 B 329 B
Overall change 650 B 650 B
Serverless bundles
vercel/next.js canary karlhorky/next.js patch-1 Change
_error.js 918 kB 918 kB
404.html 4.61 kB 4.61 kB
hooks.html 3.86 kB 3.86 kB
index.js 918 kB 918 kB
link.js 977 kB 977 kB
routerDirect.js 969 kB 969 kB
withRouter.js 969 kB 969 kB
Overall change 4.76 MB 4.76 MB
Commit: d8bf38d

@ijjk
Copy link
Member

ijjk commented Nov 18, 2020

Stats from current PR

Default Server Mode (Increase detected ⚠️)
General
vercel/next.js canary karlhorky/next.js patch-1 Change
buildDuration 9.9s 10.1s ⚠️ +177ms
nodeModulesSize 84.9 MB 84.9 MB
Page Load Tests Overall increase ✓
vercel/next.js canary karlhorky/next.js patch-1 Change
/ failed reqs 0 0
/ total time (seconds) 2.247 2.24 -0.01
/ avg req/sec 1112.5 1115.92 +3.42
/error-in-render failed reqs 0 0
/error-in-render total time (seconds) 1.295 1.281 -0.01
/error-in-render avg req/sec 1931.13 1951.39 +20.26
Client Bundles (main, webpack, commons)
vercel/next.js canary karlhorky/next.js patch-1 Change
677f882d2ed8..4972.js gzip 12.7 kB 12.7 kB
framework.HASH.js gzip 39 kB 39 kB
main-f1a49fb..e45e.js gzip 6.52 kB 6.52 kB
webpack-e067..f178.js gzip 751 B 751 B
Overall change 58.9 kB 58.9 kB
Legacy Client Bundles (polyfills)
vercel/next.js canary karlhorky/next.js patch-1 Change
polyfills-4b..e242.js gzip 31 kB 31 kB
Overall change 31 kB 31 kB
Client Pages
vercel/next.js canary karlhorky/next.js patch-1 Change
_app-3b0cf13..85f8.js gzip 1.28 kB 1.28 kB
_error-6f635..c393.js gzip 3.44 kB 3.44 kB
hooks-d4ffc3..9e0f.js gzip 887 B 887 B
index-17468f..5d83.js gzip 227 B 227 B
link-b618194..5477.js gzip 1.61 kB 1.61 kB
routerDirect..924c.js gzip 284 B 284 B
withRouter-7..c13d.js gzip 284 B 284 B
Overall change 8.01 kB 8.01 kB
Client Build Manifests
vercel/next.js canary karlhorky/next.js patch-1 Change
_buildManifest.js gzip 321 B 321 B
Overall change 321 B 321 B
Rendered Page Sizes
vercel/next.js canary karlhorky/next.js patch-1 Change
index.html gzip 613 B 613 B
link.html gzip 621 B 621 B
withRouter.html gzip 608 B 608 B
Overall change 1.84 kB 1.84 kB

Serverless Mode
General
vercel/next.js canary karlhorky/next.js patch-1 Change
buildDuration 11.5s 11.6s ⚠️ +80ms
nodeModulesSize 84.9 MB 84.9 MB
Client Bundles (main, webpack, commons)
vercel/next.js canary karlhorky/next.js patch-1 Change
677f882d2ed8..4972.js gzip 12.7 kB 12.7 kB
framework.HASH.js gzip 39 kB 39 kB
main-f1a49fb..e45e.js gzip 6.52 kB 6.52 kB
webpack-e067..f178.js gzip 751 B 751 B
Overall change 58.9 kB 58.9 kB
Legacy Client Bundles (polyfills)
vercel/next.js canary karlhorky/next.js patch-1 Change
polyfills-4b..e242.js gzip 31 kB 31 kB
Overall change 31 kB 31 kB
Client Pages
vercel/next.js canary karlhorky/next.js patch-1 Change
_app-3b0cf13..85f8.js gzip 1.28 kB 1.28 kB
_error-6f635..c393.js gzip 3.44 kB 3.44 kB
hooks-d4ffc3..9e0f.js gzip 887 B 887 B
index-17468f..5d83.js gzip 227 B 227 B
link-b618194..5477.js gzip 1.61 kB 1.61 kB
routerDirect..924c.js gzip 284 B 284 B
withRouter-7..c13d.js gzip 284 B 284 B
Overall change 8.01 kB 8.01 kB
Client Build Manifests
vercel/next.js canary karlhorky/next.js patch-1 Change
_buildManifest.js gzip 321 B 321 B
Overall change 321 B 321 B
Serverless bundles
vercel/next.js canary karlhorky/next.js patch-1 Change
_error.js 914 kB 914 kB
404.html 2.67 kB 2.67 kB
hooks.html 1.92 kB 1.92 kB
index.js 915 kB 915 kB
link.js 973 kB 973 kB
routerDirect.js 966 kB 966 kB
withRouter.js 966 kB 966 kB
Overall change 4.74 MB 4.74 MB
Commit: fa629d7

@ijjk
Copy link
Member

ijjk commented Nov 18, 2020

Stats from current PR

Default Server Mode (Increase detected ⚠️)
General
vercel/next.js canary karlhorky/next.js patch-1 Change
buildDuration 10.9s 11s ⚠️ +105ms
nodeModulesSize 84.9 MB 84.9 MB
Page Load Tests Overall increase ✓
vercel/next.js canary karlhorky/next.js patch-1 Change
/ failed reqs 0 0
/ total time (seconds) 2.605 2.54 -0.06
/ avg req/sec 959.57 984.26 +24.69
/error-in-render failed reqs 0 0
/error-in-render total time (seconds) 1.667 1.619 -0.05
/error-in-render avg req/sec 1499.51 1544.59 +45.08
Client Bundles (main, webpack, commons)
vercel/next.js canary karlhorky/next.js patch-1 Change
677f882d2ed8..4972.js gzip 12.7 kB 12.7 kB
framework.HASH.js gzip 39 kB 39 kB
main-f1a49fb..e45e.js gzip 6.52 kB 6.52 kB
webpack-e067..f178.js gzip 751 B 751 B
Overall change 58.9 kB 58.9 kB
Legacy Client Bundles (polyfills)
vercel/next.js canary karlhorky/next.js patch-1 Change
polyfills-4b..e242.js gzip 31 kB 31 kB
Overall change 31 kB 31 kB
Client Pages
vercel/next.js canary karlhorky/next.js patch-1 Change
_app-3b0cf13..85f8.js gzip 1.28 kB 1.28 kB
_error-6f635..c393.js gzip 3.44 kB 3.44 kB
hooks-d4ffc3..9e0f.js gzip 887 B 887 B
index-17468f..5d83.js gzip 227 B 227 B
link-b618194..5477.js gzip 1.61 kB 1.61 kB
routerDirect..924c.js gzip 284 B 284 B
withRouter-7..c13d.js gzip 284 B 284 B
Overall change 8.01 kB 8.01 kB
Client Build Manifests
vercel/next.js canary karlhorky/next.js patch-1 Change
_buildManifest.js gzip 321 B 321 B
Overall change 321 B 321 B
Rendered Page Sizes
vercel/next.js canary karlhorky/next.js patch-1 Change
index.html gzip 613 B 613 B
link.html gzip 621 B 621 B
withRouter.html gzip 608 B 608 B
Overall change 1.84 kB 1.84 kB

Serverless Mode
General
vercel/next.js canary karlhorky/next.js patch-1 Change
buildDuration 12.8s 12.9s ⚠️ +126ms
nodeModulesSize 84.9 MB 84.9 MB
Client Bundles (main, webpack, commons)
vercel/next.js canary karlhorky/next.js patch-1 Change
677f882d2ed8..4972.js gzip 12.7 kB 12.7 kB
framework.HASH.js gzip 39 kB 39 kB
main-f1a49fb..e45e.js gzip 6.52 kB 6.52 kB
webpack-e067..f178.js gzip 751 B 751 B
Overall change 58.9 kB 58.9 kB
Legacy Client Bundles (polyfills)
vercel/next.js canary karlhorky/next.js patch-1 Change
polyfills-4b..e242.js gzip 31 kB 31 kB
Overall change 31 kB 31 kB
Client Pages
vercel/next.js canary karlhorky/next.js patch-1 Change
_app-3b0cf13..85f8.js gzip 1.28 kB 1.28 kB
_error-6f635..c393.js gzip 3.44 kB 3.44 kB
hooks-d4ffc3..9e0f.js gzip 887 B 887 B
index-17468f..5d83.js gzip 227 B 227 B
link-b618194..5477.js gzip 1.61 kB 1.61 kB
routerDirect..924c.js gzip 284 B 284 B
withRouter-7..c13d.js gzip 284 B 284 B
Overall change 8.01 kB 8.01 kB
Client Build Manifests
vercel/next.js canary karlhorky/next.js patch-1 Change
_buildManifest.js gzip 321 B 321 B
Overall change 321 B 321 B
Serverless bundles
vercel/next.js canary karlhorky/next.js patch-1 Change
_error.js 914 kB 914 kB
404.html 2.67 kB 2.67 kB
hooks.html 1.92 kB 1.92 kB
index.js 915 kB 915 kB
link.js 973 kB 973 kB
routerDirect.js 966 kB 966 kB
withRouter.js 966 kB 966 kB
Overall change 4.74 MB 4.74 MB
Commit: 3e90856

@karlhorky
Copy link
Contributor Author

karlhorky commented Nov 18, 2020

Ok @lfades, looks like we have a thumbs-up on this setup: #18096 (comment)

@karlhorky
Copy link
Contributor Author

@lfades I've also removed the with-emotion-11 example in 32a668c, per #19233 (comment)

@ijjk
Copy link
Member

ijjk commented Nov 19, 2020

Stats from current PR

Default Server Mode (Decrease detected ✓)
General
vercel/next.js canary karlhorky/next.js patch-1 Change
buildDuration 10.4s 10.1s -284ms
nodeModulesSize 84.9 MB 84.9 MB
Page Load Tests Overall decrease ⚠️
vercel/next.js canary karlhorky/next.js patch-1 Change
/ failed reqs 0 0
/ total time (seconds) 2.266 2.25 -0.02
/ avg req/sec 1103.27 1111.34 +8.07
/error-in-render failed reqs 0 0
/error-in-render total time (seconds) 1.254 1.273 ⚠️ +0.02
/error-in-render avg req/sec 1993.03 1963.75 ⚠️ -29.28
Client Bundles (main, webpack, commons)
vercel/next.js canary karlhorky/next.js patch-1 Change
677f882d2ed8..4972.js gzip 12.7 kB 12.7 kB
framework.HASH.js gzip 39 kB 39 kB
main-f1a49fb..e45e.js gzip 6.52 kB 6.52 kB
webpack-e067..f178.js gzip 751 B 751 B
Overall change 58.9 kB 58.9 kB
Legacy Client Bundles (polyfills)
vercel/next.js canary karlhorky/next.js patch-1 Change
polyfills-4b..e242.js gzip 31 kB 31 kB
Overall change 31 kB 31 kB
Client Pages
vercel/next.js canary karlhorky/next.js patch-1 Change
_app-3b0cf13..85f8.js gzip 1.28 kB 1.28 kB
_error-6f635..c393.js gzip 3.44 kB 3.44 kB
hooks-d4ffc3..9e0f.js gzip 887 B 887 B
index-17468f..5d83.js gzip 227 B 227 B
link-b618194..5477.js gzip 1.61 kB 1.61 kB
routerDirect..924c.js gzip 284 B 284 B
withRouter-7..c13d.js gzip 284 B 284 B
Overall change 8.01 kB 8.01 kB
Client Build Manifests
vercel/next.js canary karlhorky/next.js patch-1 Change
_buildManifest.js gzip 321 B 321 B
Overall change 321 B 321 B
Rendered Page Sizes
vercel/next.js canary karlhorky/next.js patch-1 Change
index.html gzip 613 B 613 B
link.html gzip 621 B 621 B
withRouter.html gzip 608 B 608 B
Overall change 1.84 kB 1.84 kB

Serverless Mode
General
vercel/next.js canary karlhorky/next.js patch-1 Change
buildDuration 12.2s 11.7s -441ms
nodeModulesSize 84.9 MB 84.9 MB
Client Bundles (main, webpack, commons)
vercel/next.js canary karlhorky/next.js patch-1 Change
677f882d2ed8..4972.js gzip 12.7 kB 12.7 kB
framework.HASH.js gzip 39 kB 39 kB
main-f1a49fb..e45e.js gzip 6.52 kB 6.52 kB
webpack-e067..f178.js gzip 751 B 751 B
Overall change 58.9 kB 58.9 kB
Legacy Client Bundles (polyfills)
vercel/next.js canary karlhorky/next.js patch-1 Change
polyfills-4b..e242.js gzip 31 kB 31 kB
Overall change 31 kB 31 kB
Client Pages
vercel/next.js canary karlhorky/next.js patch-1 Change
_app-3b0cf13..85f8.js gzip 1.28 kB 1.28 kB
_error-6f635..c393.js gzip 3.44 kB 3.44 kB
hooks-d4ffc3..9e0f.js gzip 887 B 887 B
index-17468f..5d83.js gzip 227 B 227 B
link-b618194..5477.js gzip 1.61 kB 1.61 kB
routerDirect..924c.js gzip 284 B 284 B
withRouter-7..c13d.js gzip 284 B 284 B
Overall change 8.01 kB 8.01 kB
Client Build Manifests
vercel/next.js canary karlhorky/next.js patch-1 Change
_buildManifest.js gzip 321 B 321 B
Overall change 321 B 321 B
Serverless bundles
vercel/next.js canary karlhorky/next.js patch-1 Change
_error.js 914 kB 914 kB
404.html 2.67 kB 2.67 kB
hooks.html 1.92 kB 1.92 kB
index.js 915 kB 915 kB
link.js 973 kB 973 kB
routerDirect.js 966 kB 966 kB
withRouter.js 966 kB 966 kB
Overall change 4.74 MB 4.74 MB
Commit: 32a668c

@vercel
Copy link

vercel bot commented Nov 19, 2020

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/vercel/nextjs-examples-image-component/esejtio5i
✅ Preview: https://nextjs-examples-image-component-git-patch-1.vercel.vercel.app

@vercel vercel bot temporarily deployed to Preview November 19, 2020 22:46 Inactive
Copy link
Member

@lfades lfades left a comment

Choose a reason for hiding this comment

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

Added some minor updates and it looks good to me! Thank you @karlhorky 💯 🔥

@vercel vercel bot temporarily deployed to Preview November 19, 2020 22:49 Inactive
@kodiakhq kodiakhq bot merged commit 65db069 into vercel:canary Nov 19, 2020
@karlhorky
Copy link
Contributor Author

Great, glad to help! Thanks for your patience and the merge @lfades !

@karlhorky karlhorky deleted the patch-1 branch November 20, 2020 09:54
kamermans pushed a commit to kamermans/next.js that referenced this pull request Dec 14, 2020
@vercel vercel locked as resolved and limited conversation to collaborators Jan 29, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
examples Issue was opened via the examples template.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants