Skip to content

[pull] master from microsoft:master#24

Merged
pull[bot] merged 3 commits intoksti:masterfrom
microsoft:master
Oct 29, 2019
Merged

[pull] master from microsoft:master#24
pull[bot] merged 3 commits intoksti:masterfrom
microsoft:master

Conversation

@pull
Copy link
Copy Markdown

@pull pull bot commented Oct 29, 2019

See Commits and Changes for more details.


Created by pull[bot]. Want to support this open source service? Please star it : )

TypeScript Bot and others added 3 commits October 29, 2019 14:26
* resolve require with entity name postfix

For example, `require("x").c`. This is the value equivalent of
`import("x").a.b.c`, but the syntax tree is not as nicely designed for
this purpose.

Fixes #34802

* Add bug number to test

* Add optional chain test
* Add isIntersectionConstituent to relation key

isIntersectionConstituent controls whether relation checking performs
excess property and common property checks. It is possible to fail a
relation check with excess property checks turned on, cache the result,
and then skip a relation check with excess property checks that would
have succeeded. #33133 provides an example of such a program.

Fixes #33133 the right way, so I reverted the fix at #33213
Fixes #34762 (by reverting #33213)
Fixes #33944 -- I added the test from #34646

* Update comments in test
@pull pull bot added the ⤵️ pull label Oct 29, 2019
@pull pull bot merged commit 00dd1f0 into ksti:master Oct 29, 2019
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.

1 participant