-
Notifications
You must be signed in to change notification settings - Fork 12.7k
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
Ignore any
in contextual intersection types
#59528
Conversation
@typescript-bot test it |
Hey @ahejlsberg, the results of running the DT tests are ready. There were interesting changes: Branch only errors:Package: mdx
Package: jquery
|
This PR breaks the following test cases: TS playground. All of them are handled without introducing (known 😉 ) regressions by #52095 . I know that this PR looks more complex than what's needed. A good chunk of those tests were discovered this week by user/topXXX runs for my PR and I was working hard on fixing them in the past days and adding some extra tests for regressions that were not caught by those extended test runs. At one point of this PR, I had a version similar-ish to the one here. However, the current branch that handles |
@ahejlsberg Here they are:
tscComparison Report - baseline..pr
System info unknown
Hosts
Scenarios
Developer Information: |
@ahejlsberg Here are the results of running the top 400 repos with tsc comparing Something interesting changed - please have a look. Details
|
Closing in favor of #52095. |
Implements what I describe here.
Fixes #48812.
Fixes #55150.
Fixes #59473.