-
Notifications
You must be signed in to change notification settings - Fork 12.6k
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
Issue better error when unresolvable package in --moduleResolution node10
resolves with --moduleResolution bundler
#56949
Conversation
@sheetalkamat I'm not sure whether this is safe with the existing |
@typescript-bot perf test |
Heya @andrewbranch, I've started to run the regular perf test suite on this PR at 7383586. You can monitor the build here. Update: The results are in! |
@andrewbranch Here they are:
startupComparison Report - baseline..pr
System info unknown
Hosts
Scenarios
tscComparison Report - baseline..pr
System info unknown
Hosts
Scenarios
tsserverComparison Report - baseline..pr
System info unknown
Hosts
Scenarios
Developer Information: |
@typescript-bot perf test |
Heya @andrewbranch, I've started to run the public perf test suite on this PR at 599f1ba. You can monitor the build here. Update: The results are in! |
Heya @andrewbranch, I've started to run the regular perf test suite on this PR at 599f1ba. You can monitor the build here. Update: The results are in! |
@andrewbranch Here they are:
tscComparison Report - baseline..pr
System info unknown
Hosts
Scenarios
Developer Information: |
@andrewbranch Here they are:
startupComparison Report - baseline..pr
System info unknown
Hosts
Scenarios
tscComparison Report - baseline..pr
System info unknown
Hosts
Scenarios
tsserverComparison Report - baseline..pr
System info unknown
Hosts
Scenarios
Developer Information: |
Fixes #55104