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

perf: change build target to es2018 #1423

Merged
merged 5 commits into from
Jun 1, 2020
Merged

perf: change build target to es2018 #1423

merged 5 commits into from
Jun 1, 2020

Conversation

merceyz
Copy link
Member

@merceyz merceyz commented May 31, 2020

What's the problem this PR addresses?

We currently support node >=10.19.0, but we're building for v9.11.2 causing us to include unnecessary helpers which are slower than what node natively supports.

How did you fix it?

Changed the build target of TypeScript to ES2018 which according to https://node.green/ is supported in v10.3.0 and higher. Except for some proxy stuff but we don't use that anyways.

Result
Table of size differences in bytes

before after reduction
pnp hook 163 353 163 209 44
CLI 2 505 541 2 498 267 7 274

Checklist

  • I have read the Contributing Guide.
  • I have set the packages that need to be released for my changes to be effective.
  • I have verified that all automated PR checks pass.

@merceyz merceyz requested a review from arcanis as a code owner May 31, 2020 13:25
@arcanis arcanis merged commit 6dda734 into master Jun 1, 2020
@arcanis arcanis deleted the merceyz/es2018 branch June 1, 2020 08:37
@merceyz merceyz mentioned this pull request Jun 12, 2020
3 tasks
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 this pull request may close these issues.

2 participants