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: support "lib: es6" #850

Merged
merged 1 commit into from
Jun 13, 2023
Merged

feat: support "lib: es6" #850

merged 1 commit into from
Jun 13, 2023

Conversation

f0rmat1k
Copy link
Contributor

@f0rmat1k f0rmat1k commented Jun 13, 2023

es2017 doesn't require array length argument. But 2015 and below needs it https://github.com/microsoft/TypeScript/blob/2a37eb2e471e01e510849f0dc108b3699ab869db/src/lib/es2015.iterable.d.ts#L268

Anyway according documentation zero – is default initializer https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Uint8Array

So if you have lib param in tsconfig below es2017, you've got a problem with typechecking.
I just replaced every empty call to (0). It must work on any es lib configuration.

@f0rmat1k f0rmat1k changed the title feat: support lib: es6 feat: support "lib: es6" Jun 13, 2023
@stephenh
Copy link
Owner

Cool, thanks @f0rmat1k !

@stephenh stephenh merged commit 6280677 into stephenh:main Jun 13, 2023
13 of 15 checks passed
stephenh pushed a commit that referenced this pull request Jun 13, 2023
# [1.149.0](v1.148.2...v1.149.0) (2023-06-13)

### Features

* support lib: es6 ([#850](#850)) ([6280677](6280677))
@stephenh
Copy link
Owner

🎉 This PR is included in version 1.149.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

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.

None yet

2 participants