Skip to content

worker-thread amp binary: add in missing compiler plugin#885

Closed
samouri wants to merge 1 commit intoampproject:mainfrom
samouri:missing-compile
Closed

worker-thread amp binary: add in missing compiler plugin#885
samouri wants to merge 1 commit intoampproject:mainfrom
samouri:missing-compile

Conversation

@samouri
Copy link
Copy Markdown
Member

@samouri samouri commented Jul 20, 2020

Should we be running the AMP version through terser? If so then I'll also add this to main-thread.

before: mjs was +17kb
before

after: mjs is -1.62kb
after

@samouri samouri requested a review from dreamofabear July 20, 2020 19:34
@samouri samouri self-assigned this Jul 20, 2020
@google-cla google-cla bot added the cla: yes label Jul 20, 2020
@samouri samouri changed the title Worker-thread amp binary: add in missing compile plugins worker-thread amp binary: add in missing compiler plugin Jul 20, 2020
@samouri samouri mentioned this pull request Jul 20, 2020
@samouri
Copy link
Copy Markdown
Member Author

samouri commented Jul 20, 2020

Closing as this was intentional: #882 (comment)

@samouri samouri closed this Jul 20, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant