X Tutup
The Wayback Machine - https://web.archive.org/web/20210410083327/https://github.com/microsoft/TypeScript/pull/41849
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

[release-4.1] Undo #39258 (isArray changes) - see overview at #41808 #41849

Merged
merged 1 commit into from Dec 7, 2020

Conversation

@orta
Copy link
Member

@orta orta commented Dec 7, 2020

In our latest design meeting we concluded that it was better to revert the isArray changes in #39258 - we'll probably come back to this with #36554.

Fixes #41808 by removing the .d.ts change. We think the assignability check is still worth keeping in.

@DanielRosenwasser DanielRosenwasser changed the title Undo #39258 (isArray changes) see overview at #41808 for 4.1 [release-4.1] Undo #39258 (isArray changes) - see overview at #41808 Dec 7, 2020
@orta orta merged commit abf8815 into microsoft:release-4.1 Dec 7, 2020
10 checks passed
10 checks passed
build (10.x)
Details
CodeQL-Build CodeQL-Build
Details
build (12.x)
Details
build (14.x)
Details
CodeQL No new alerts
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
license/cla All CLA requirements met.
Details
node10 Build #90537 succeeded
Details
node12 Build #90535 succeeded
Details
node14 Build #90536 succeeded
Details
@lukeapage
Copy link

@lukeapage lukeapage commented Dec 12, 2020

Will you reopen #17002 which is now broken again in 4.1.3 (had to re ignore new errors when upgrading)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

5 participants
X Tutup