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 webpack polyfills (#9735) #9738

Merged
merged 1 commit into from Jan 12, 2020

Conversation

@silverwind
Copy link
Member

silverwind commented Jan 12, 2020

Backport of #9735

webpack polyfills did not work because useBuiltIns: 'entry' expects a
explicit core-js import. Changed it to 'usage' which does not require
these explicit imports and polyfills based on browserslist.

As a result, the built index.js now went from 128kB to 192kB.

Ref: https://babeljs.io/docs/en/babel-preset-env#usebuiltins

Co-authored-by: zeripath <art27@cantab.net>
@lafriks lafriks added the kind/build label Jan 12, 2020
@lafriks lafriks added this to the 1.11.0 milestone Jan 12, 2020
@GiteaBot GiteaBot added lgtm/done and removed lgtm/need 1 labels Jan 12, 2020
@techknowlogick techknowlogick merged commit 3e23dad into go-gitea:release/v1.11 Jan 12, 2020
2 checks passed
2 checks passed
approvals/lgtm this commit looks good
continuous-integration/drone/pr Build is passing
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
4 participants
You can’t perform that action at this time.