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

feat(gatsby): Speedup building Functions by disabling minification #31473

Merged
merged 1 commit into from
May 19, 2021

Conversation

KyleAMathews
Copy link
Contributor

This speeds up building Functions in the functions integration test from 2.3s to 0.65s.

Minification is much more useful for preparing JS for browsers vs. serverless functions as many browsers are running on slow devices on slow networks so we want to make JS bundles as small as possible.

Serverless functions, on the other hand, are loaded by servers over very fast networks so the extra CPU/memory costs to minify isn't worth it.

This speeds up building Functions in the functions integration test from 2.3s to 0.65s.

Minification is much more useful for preparing JS for browsers vs. serverless functions as many browsers are running on slow devices on slow networks so we want to make JS bundles as small as possible.

Serverless functions, on the other hand, are loaded by servers over very fast networks so the extra CPU/memory costs to minify isn't worth it.
@gatsbot gatsbot bot added the status: triage needed Issue or pull request that need to be triaged and assigned to a reviewer label May 19, 2021
@KyleAMathews KyleAMathews added topic: functions and removed status: triage needed Issue or pull request that need to be triaged and assigned to a reviewer labels May 19, 2021
@wardpeet wardpeet merged commit a8a4f76 into master May 19, 2021
@wardpeet wardpeet deleted the disable-minification branch May 19, 2021 08:57
@LekoArts LekoArts added the topic: DX Developer Experience (e.g. Fast Refresh, i18n, SSR, page creation, starters) label May 28, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
topic: DX Developer Experience (e.g. Fast Refresh, i18n, SSR, page creation, starters)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants