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

Fix chunking tests failing from timeout being too low #8360

Merged
merged 2 commits into from Aug 14, 2019

Conversation

ijjk
Copy link
Member

@ijjk ijjk commented Aug 14, 2019

Noticed this suite was failing in Azure for a bit from the default timeout not being high enough

@ijjk
Copy link
Member Author

ijjk commented Aug 14, 2019

Stats from current PR

Default Server Mode
General
zeit/next.js canary ijjk/next.js fix/chunking-tests-windows Change
buildDuration 22.2s 22.5s ⚠️ +283ms
nodeModulesSize 41.2 MB 41.2 MB
Client Bundles (main, webpack, commons)
zeit/next.js canary ijjk/next.js fix/chunking-tests-windows Change
main-HASH.js 13.8 kB 13.8 kB
main-HASH.js gzip 4.87 kB 4.87 kB
webpack-HASH.js 1.53 kB 1.53 kB
webpack-HASH.js gzip 746 B 746 B
commons.HASH.js 194 kB 194 kB
commons.HASH.js gzip 63.1 kB 63.1 kB
Overall change
Client Bundles (main, webpack, commons) Modern
zeit/next.js canary ijjk/next.js fix/chunking-tests-windows Change
main-HASH.module.js 11.4 kB 11.4 kB
main-HASH.module.js gzip 4.31 kB 4.31 kB
webpack-HASH.module.js 1.53 kB 1.53 kB
webpack-HASH.module.js gzip 746 B 746 B
commons.HASH.module.js 175 kB 175 kB
commons.HASH.module.js gzip 57.1 kB 57.1 kB
Overall change
Client Pages
zeit/next.js canary ijjk/next.js fix/chunking-tests-windows Change
_app.js 1.9 kB 1.9 kB
_app.js gzip 905 B 905 B
_error.js 8.02 kB 8.02 kB
_error.js gzip 3.12 kB 3.12 kB
index.js 343 B 343 B
index.js gzip 237 B 237 B
link.js 4.07 kB 4.07 kB
link.js gzip 1.79 kB 1.79 kB
routerDirect.js 423 B 423 B
routerDirect.js gzip 290 B 290 B
withRouter.js 435 B 435 B
withRouter.js gzip 287 B 287 B
Overall change
Client Pages Modern
zeit/next.js canary ijjk/next.js fix/chunking-tests-windows Change
_app.module.js 1.79 kB 1.79 kB
_app.module.js gzip 865 B 865 B
_error.module.js 5.82 kB 5.82 kB
_error.module.js gzip 2.33 kB 2.33 kB
index.module.js 319 B 319 B
index.module.js gzip 238 B 238 B
link.module.js 3.78 kB 3.78 kB
link.module.js gzip 1.7 kB 1.7 kB
routerDirect.module.js 411 B 411 B
routerDirect.module.js gzip 291 B 291 B
withRouter.module.js 423 B 423 B
withRouter.module.js gzip 288 B 288 B
Overall change
Fetched pages
zeit/next.js canary ijjk/next.js fix/chunking-tests-windows Change
link 2.81 kB 2.81 kB
link gzip 847 B 844 B -3 B
index 2.76 kB 2.76 kB
index gzip 836 B 837 B ⚠️ +1 B
withRouter 2.82 kB 2.82 kB
withRouter gzip 835 B 830 B -5 B
Overall change

Serverless Mode
General
zeit/next.js canary ijjk/next.js fix/chunking-tests-windows Change
buildDuration 24.1s 24.3s ⚠️ +209ms
nodeModulesSize 41.2 MB 41.2 MB
Client Bundles (main, webpack, commons)
zeit/next.js canary ijjk/next.js fix/chunking-tests-windows Change
main-HASH.js 13.8 kB 13.8 kB
main-HASH.js gzip 4.87 kB 4.87 kB
webpack-HASH.js 1.53 kB 1.53 kB
webpack-HASH.js gzip 746 B 746 B
commons.HASH.js 194 kB 194 kB
commons.HASH.js gzip 63.1 kB 63.1 kB
Overall change
Client Bundles (main, webpack, commons) Modern
zeit/next.js canary ijjk/next.js fix/chunking-tests-windows Change
main-HASH.module.js 11.4 kB 11.4 kB
main-HASH.module.js gzip 4.31 kB 4.31 kB
webpack-HASH.module.js 1.53 kB 1.53 kB
webpack-HASH.module.js gzip 746 B 746 B
commons.HASH.module.js 175 kB 175 kB
commons.HASH.module.js gzip 57.1 kB 57.1 kB
Overall change
Client Pages
zeit/next.js canary ijjk/next.js fix/chunking-tests-windows Change
_app.js 1.9 kB 1.9 kB
_app.js gzip 905 B 905 B
_error.js 8.02 kB 8.02 kB
_error.js gzip 3.12 kB 3.12 kB
index.js 343 B 343 B
index.js gzip 237 B 237 B
link.js 4.07 kB 4.07 kB
link.js gzip 1.79 kB 1.79 kB
routerDirect.js 423 B 423 B
routerDirect.js gzip 290 B 290 B
withRouter.js 435 B 435 B
withRouter.js gzip 287 B 287 B
Overall change
Client Pages Modern
zeit/next.js canary ijjk/next.js fix/chunking-tests-windows Change
_app.module.js 1.79 kB 1.79 kB
_app.module.js gzip 865 B 865 B
_error.module.js 5.82 kB 5.82 kB
_error.module.js gzip 2.33 kB 2.33 kB
index.module.js 319 B 319 B
index.module.js gzip 238 B 238 B
link.module.js 3.78 kB 3.78 kB
link.module.js gzip 1.7 kB 1.7 kB
routerDirect.module.js 411 B 411 B
routerDirect.module.js gzip 291 B 291 B
withRouter.module.js 423 B 423 B
withRouter.module.js gzip 288 B 288 B
Overall change
Serverless bundles
zeit/next.js canary ijjk/next.js fix/chunking-tests-windows Change
_error.js 247 kB 247 kB
_error.js gzip 66.2 kB 66.2 kB ⚠️ +2 B
index.js 247 kB 247 kB
index.js gzip 66.5 kB 66.5 kB ⚠️ +1 B
link.js 255 kB 255 kB
link.js gzip 68.7 kB 68.7 kB ⚠️ +1 B
routerDirect.js 248 kB 248 kB
routerDirect.js gzip 66.4 kB 66.4 kB ⚠️ +1 B
withRouter.js 248 kB 248 kB
withRouter.js gzip 66.5 kB 66.5 kB ⚠️ +2 B
Overall change

@Timer Timer merged commit 7c09f71 into vercel:canary Aug 14, 2019
@ijjk ijjk deleted the fix/chunking-tests-windows branch August 14, 2019 01:54
@vercel vercel locked as resolved and limited conversation to collaborators Feb 1, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants