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

[Master feature] Leaner AMP #14392

Closed
dreamofabear opened this issue Apr 3, 2018 · 6 comments · Fixed by #15256
Closed

[Master feature] Leaner AMP #14392

dreamofabear opened this issue Apr 3, 2018 · 6 comments · Fixed by #15256
Assignees

Comments

@dreamofabear
Copy link

dreamofabear commented Apr 3, 2018

Goal: Reduce size of AMP’s main JS binary by 50%.
Corollary: Do not regress parse, compile, and execution time of the JS.

@dreamofabear
Copy link
Author

Work for Q2 2018:

  • Automated binary size tracking & dashboard
  • Deliver module/no-module versions of AMP main JS binary
  • Bundle analysis and strategy planning
  • Reduce binary size by 10%

@dreamofabear
Copy link
Author

Bundle analyses from April 3, 2018:

@dreamofabear
Copy link
Author

Note: Individual tasks tracked in Build Size project.

@dreamofabear
Copy link
Author

Accidental close.

@dreamofabear dreamofabear reopened this May 15, 2018
@rsimha
Copy link
Contributor

rsimha commented May 15, 2018

Sheesh, Required in order to fix closed this :/

@ampprojectbot ampprojectbot added this to the Pending Triage milestone Jun 5, 2018
@kristoferbaxter
Copy link
Contributor

Closed in favor of #16745.

jridgewell added a commit to jridgewell/amphtml that referenced this issue Aug 30, 2018
```
[16:36:45] Finished 'dist' after 2.5 min
[16:36:49] Using gulpfile ~/src/amphtml/gulpfile.js
[16:36:49] Starting 'bundle-size'...
[16:36:49] Running npx bundlesize -f "./dist/v0.js" -s "81.15KB"...
[16:36:49] FAIL  ./dist/v0.js: 81.18KB > maxSize 81.15KB (gzip)
[16:36:49] ERROR: bundlesize found that ./dist/v0.js has exceeded its size cap of 81.15KB.
[16:36:49] This is part of a new effort to reduce AMP's binary size (ampproject#14392).
[16:36:49] Please contact @choumx or @jridgewell for assistance.
[16:36:49] Finished 'bundle-size' after 498 ms
```
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants