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: target es2019 in our tsconfig #7967

Closed
wants to merge 2 commits into from

Conversation

imnotjames
Copy link
Contributor

@imnotjames imnotjames commented Jul 28, 2021

Description of change

this enables a number of features that tsconfig can use
when compiling to javascript - including native promises
and Array.includes(). however, this does limit us to
node 12+

  • target: es2019 - Node 12+ supports syntax
  • lib: es2020 - Node 12+ supports the standard library

fixes #7223

Pull-Request Checklist

  • Code is up-to-date with the master branch
  • npm run lint passes with this change
  • npm run test passes with this change
  • This pull request links relevant issues as Fixes #0000
  • There are new or updated unit tests validating the change
  • Documentation has been updated to reflect this change
  • The new commits follow conventions explained in CONTRIBUTING.md

this enables a number of features that tsconfig can use
when compiling to javascript - including native promises
and `Array.includes()`.  however, this does limit us to
node 12+
@imnotjames imnotjames mentioned this pull request Jul 28, 2021
@imnotjames imnotjames marked this pull request as ready for review July 29, 2021 02:51
@pleerock pleerock added this to the 0.3.0 milestone Nov 8, 2021
@@ -1,10 +1,10 @@
{
"compilerOptions": {
"lib": ["es5", "es6"],
"lib": ["es2020"],
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Commit message says "2019", but it's "2020" here.

@pleerock
Copy link
Member

pleerock commented Feb 9, 2022

Was merged into #8616

@pleerock pleerock closed this Feb 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Having typeorm compiled for es6 or above.
3 participants