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

Use contextual type to determine 'this' when determining member visibility #56105

Merged
merged 5 commits into from
Aug 2, 2024

Conversation

RyanCavanaugh
Copy link
Member

Fixes #56094

@jakebailey
Copy link
Member

@typescript-bot test top200
@typescript-bot user test this
@typescript-bot run dt
@typescript-bot perf test this
@typescript-bot pack this

@typescript-bot
Copy link
Collaborator

typescript-bot commented Oct 13, 2023

Heya @jakebailey, I've started to run the regular perf test suite on this PR at 215a87a. You can monitor the build here.

Update: The results are in!

@typescript-bot
Copy link
Collaborator

typescript-bot commented Oct 13, 2023

Heya @jakebailey, I've started to run the tarball bundle task on this PR at 215a87a. You can monitor the build here.

@typescript-bot
Copy link
Collaborator

typescript-bot commented Oct 13, 2023

Heya @jakebailey, I've started to run the parallelized Definitely Typed test suite on this PR at 215a87a. You can monitor the build here.

Update: The results are in!

@typescript-bot
Copy link
Collaborator

typescript-bot commented Oct 13, 2023

Heya @jakebailey, I've started to run the diff-based top-repos suite on this PR at 215a87a. You can monitor the build here.

Update: The results are in!

@typescript-bot
Copy link
Collaborator

typescript-bot commented Oct 13, 2023

Heya @jakebailey, I've started to run the diff-based user code test suite on this PR at 215a87a. You can monitor the build here.

Update: The results are in!

@typescript-bot
Copy link
Collaborator

typescript-bot commented Oct 13, 2023

Hey @jakebailey, I've packed this into an installable tgz. You can install it for testing by referencing it in your package.json like so:

{
    "devDependencies": {
        "typescript": "https://typescript.visualstudio.com/cf7ac146-d525-443c-b23c-0d58337efebc/_apis/build/builds/158213/artifacts?artifactName=tgz&fileId=2B01F3860C4B0C5B3A6229C8DF6BFAA5BC2A8EA40AEA4C5568770C359EFD3E5702&fileName=/typescript-5.3.0-insiders.20231013.tgz"
    }
}

and then running npm install.


There is also a playground for this build and an npm module you can use via "typescript": "npm:@typescript-deploys/[email protected]".;

@typescript-bot
Copy link
Collaborator

@jakebailey Here are the results of running the user test suite comparing main and refs/pull/56105/merge:

There were infrastructure failures potentially unrelated to your change:

  • 3 instances of "Package install failed"
  • 1 instance of "Unknown failure"

Otherwise...

Everything looks good!

@typescript-bot
Copy link
Collaborator

@jakebailey
The results of the perf run you requested are in!

Here they are:

Compiler

Comparison Report - baseline..pr
Metric baseline pr Delta Best Worst p-value
Angular - node (v18.15.0, x64)
Memory used 295,071k (± 0.01%) 295,099k (± 0.01%) ~ 295,073k 295,127k p=0.196 n=6
Parse Time 2.63s (± 0.61%) 2.64s (± 0.28%) ~ 2.63s 2.65s p=0.282 n=6
Bind Time 0.84s (± 0.97%) 0.84s (± 0.97%) ~ 0.83s 0.85s p=1.000 n=6
Check Time 8.06s (± 0.47%) 8.06s (± 0.22%) ~ 8.03s 8.08s p=0.809 n=6
Emit Time 7.05s (± 0.47%) 7.04s (± 0.15%) ~ 7.03s 7.06s p=0.416 n=6
Total Time 18.59s (± 0.27%) 18.59s (± 0.08%) ~ 18.56s 18.60s p=1.000 n=6
Compiler-Unions - node (v18.15.0, x64)
Memory used 193,529k (± 1.64%) 193,044k (± 1.51%) ~ 190,614k 196,713k p=0.936 n=6
Parse Time 1.35s (± 0.61%) 1.35s (± 0.56%) ~ 1.34s 1.36s p=0.729 n=6
Bind Time 0.73s (± 0.00%) 0.73s (± 0.00%) ~ 0.73s 0.73s p=1.000 n=6
Check Time 9.11s (± 0.39%) 9.14s (± 0.31%) ~ 9.11s 9.18s p=0.106 n=6
Emit Time 2.64s (± 0.95%) 2.64s (± 0.54%) ~ 2.62s 2.66s p=0.570 n=6
Total Time 13.82s (± 0.41%) 13.86s (± 0.15%) ~ 13.83s 13.88s p=0.065 n=6
Monaco - node (v18.15.0, x64)
Memory used 347,299k (± 0.00%) 347,319k (± 0.00%) ~ 347,295k 347,335k p=0.065 n=6
Parse Time 2.45s (± 0.36%) 2.46s (± 0.65%) ~ 2.44s 2.48s p=0.360 n=6
Bind Time 0.94s (± 0.43%) 0.94s (± 0.00%) ~ 0.94s 0.94s p=0.405 n=6
Check Time 6.91s (± 0.54%) 6.93s (± 0.55%) ~ 6.88s 6.96s p=0.685 n=6
Emit Time 4.02s (± 0.26%) 4.02s (± 0.44%) ~ 4.00s 4.04s p=0.801 n=6
Total Time 14.32s (± 0.25%) 14.34s (± 0.28%) ~ 14.29s 14.39s p=0.470 n=6
TFS - node (v18.15.0, x64)
Memory used 302,554k (± 0.01%) 302,556k (± 0.01%) ~ 302,536k 302,597k p=0.748 n=6
Parse Time 2.01s (± 0.93%) 1.99s (± 0.49%) -0.03s (- 1.32%) 1.98s 2.00s p=0.033 n=6
Bind Time 1.00s (± 1.03%) 1.00s (± 1.21%) ~ 0.99s 1.02s p=1.000 n=6
Check Time 6.27s (± 0.32%) 6.26s (± 0.38%) ~ 6.23s 6.30s p=0.466 n=6
Emit Time 3.56s (± 0.59%) 3.57s (± 0.34%) ~ 3.55s 3.58s p=0.514 n=6
Total Time 12.85s (± 0.21%) 12.81s (± 0.16%) ~ 12.79s 12.84s p=0.061 n=6
material-ui - node (v18.15.0, x64)
Memory used 470,505k (± 0.01%) 470,494k (± 0.00%) ~ 470,472k 470,514k p=0.689 n=6
Parse Time 2.58s (± 0.47%) 2.58s (± 0.32%) ~ 2.57s 2.59s p=0.175 n=6
Bind Time 1.00s (± 1.09%) 1.00s (± 0.75%) ~ 0.99s 1.01s p=0.604 n=6
Check Time 16.62s (± 0.60%) 16.59s (± 0.45%) ~ 16.51s 16.70s p=0.688 n=6
Emit Time 0.00s (± 0.00%) 0.00s (± 0.00%) ~ 0.00s 0.00s p=1.000 n=6
Total Time 20.20s (± 0.48%) 20.17s (± 0.38%) ~ 20.10s 20.30s p=0.688 n=6
xstate - node (v18.15.0, x64)
Memory used 512,670k (± 0.02%) 512,651k (± 0.01%) ~ 512,578k 512,706k p=0.936 n=6
Parse Time 3.26s (± 0.23%) 3.26s (± 0.32%) ~ 3.25s 3.28s p=0.931 n=6
Bind Time 1.55s (± 0.26%) 1.55s (± 0.71%) ~ 1.53s 1.56s p=1.000 n=6
Check Time 2.88s (± 0.69%) 2.85s (± 0.95%) ~ 2.81s 2.88s p=0.063 n=6
Emit Time 0.08s (± 0.00%) 0.08s (± 0.00%) ~ 0.08s 0.08s p=1.000 n=6
Total Time 7.77s (± 0.18%) 7.73s (± 0.39%) -0.03s (- 0.41%) 7.70s 7.76s p=0.033 n=6
System info unknown
Hosts
  • node (v18.15.0, x64)
Scenarios
  • Angular - node (v18.15.0, x64)
  • Compiler-Unions - node (v18.15.0, x64)
  • Monaco - node (v18.15.0, x64)
  • TFS - node (v18.15.0, x64)
  • material-ui - node (v18.15.0, x64)
  • xstate - node (v18.15.0, x64)
Benchmark Name Iterations
Current pr 6
Baseline baseline 6

tsserver

Comparison Report - baseline..pr
Metric baseline pr Delta Best Worst p-value
Compiler-UnionsTSServer - node (v18.15.0, x64)
Req 1 - updateOpen 2,377ms (± 1.63%) 2,351ms (± 1.09%) ~ 2,329ms 2,399ms p=0.230 n=6
Req 2 - geterr 5,343ms (± 1.63%) 5,408ms (± 1.59%) ~ 5,280ms 5,474ms p=0.173 n=6
Req 3 - references 328ms (± 0.80%) 329ms (± 1.49%) ~ 325ms 337ms p=0.935 n=6
Req 4 - navto 279ms (± 1.51%) 276ms (± 1.38%) ~ 273ms 282ms p=0.193 n=6
Req 5 - completionInfo count 1,356 (± 0.00%) 1,356 (± 0.00%) ~ 1,356 1,356 p=1.000 n=6
Req 5 - completionInfo 80ms (±10.05%) 85ms (± 9.10%) ~ 75ms 90ms p=0.190 n=6
CompilerTSServer - node (v18.15.0, x64)
Req 1 - updateOpen 2,476ms (± 1.62%) 2,487ms (± 1.13%) ~ 2,458ms 2,539ms p=0.810 n=6
Req 2 - geterr 4,129ms (± 2.00%) 4,082ms (± 1.55%) ~ 4,045ms 4,210ms p=0.468 n=6
Req 3 - references 338ms (± 1.80%) 342ms (± 1.27%) ~ 333ms 344ms p=0.181 n=6
Req 4 - navto 284ms (± 0.73%) 284ms (± 0.36%) ~ 283ms 286ms p=0.738 n=6
Req 5 - completionInfo count 1,518 (± 0.00%) 1,518 (± 0.00%) ~ 1,518 1,518 p=1.000 n=6
Req 5 - completionInfo 82ms (± 7.15%) 85ms (± 5.42%) ~ 76ms 88ms p=0.304 n=6
xstateTSServer - node (v18.15.0, x64)
Req 1 - updateOpen 2,595ms (± 0.49%) 2,583ms (± 0.62%) ~ 2,567ms 2,604ms p=0.149 n=6
Req 2 - geterr 1,677ms (± 2.16%) 1,677ms (± 1.76%) ~ 1,634ms 1,711ms p=0.810 n=6
Req 3 - references 109ms (± 7.48%) 119ms (± 9.01%) ~ 105ms 128ms p=0.285 n=6
Req 4 - navto 359ms (± 0.34%) 359ms (± 0.45%) ~ 358ms 362ms p=0.933 n=6
Req 5 - completionInfo count 2,073 (± 0.00%) 2,073 (± 0.00%) ~ 2,073 2,073 p=1.000 n=6
Req 5 - completionInfo 302ms (± 2.01%) 306ms (± 1.79%) ~ 295ms 310ms p=0.329 n=6
System info unknown
Hosts
  • node (v18.15.0, x64)
Scenarios
  • CompilerTSServer - node (v18.15.0, x64)
  • Compiler-UnionsTSServer - node (v18.15.0, x64)
  • xstateTSServer - node (v18.15.0, x64)
Benchmark Name Iterations
Current pr 6
Baseline baseline 6

Startup

Comparison Report - baseline..pr
Metric baseline pr Delta Best Worst p-value
tsc-startup - node (v18.15.0, x64)
Execution time 152.28ms (± 0.18%) 152.22ms (± 0.18%) -0.06ms (- 0.04%) 151.25ms 156.43ms p=0.022 n=600
tsserver-startup - node (v18.15.0, x64)
Execution time 227.84ms (± 0.16%) 227.69ms (± 0.17%) -0.16ms (- 0.07%) 226.30ms 235.38ms p=0.000 n=600
tsserverlibrary-startup - node (v18.15.0, x64)
Execution time 228.85ms (± 0.17%) 228.81ms (± 0.16%) ~ 227.37ms 233.27ms p=0.261 n=600
typescript-startup - node (v18.15.0, x64)
Execution time 228.72ms (± 0.16%) 228.84ms (± 0.16%) +0.12ms (+ 0.05%) 227.37ms 234.47ms p=0.002 n=600
System info unknown
Hosts
  • node (v18.15.0, x64)
Scenarios
  • tsc-startup - node (v18.15.0, x64)
  • tsserver-startup - node (v18.15.0, x64)
  • tsserverlibrary-startup - node (v18.15.0, x64)
  • typescript-startup - node (v18.15.0, x64)
Benchmark Name Iterations
Current pr 6
Baseline baseline 6

Developer Information:

Download Benchmarks

@typescript-bot
Copy link
Collaborator

Hey @jakebailey, the results of running the DT tests are ready.
Everything looks the same!
You can check the log here.

@typescript-bot
Copy link
Collaborator

@jakebailey Here are the results of running the top-repos suite comparing main and refs/pull/56105/merge:

Everything looks good!

}
}
else {
// 3. This 'this' parameter of a contextual type
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
// 3. This 'this' parameter of a contextual type
// 3. The 'this' parameter of a contextual type

@sandersn
Copy link
Member

sandersn commented Dec 4, 2023

@RyanCavanaugh this looks like it's ready to merge. Is that right?

@RyanCavanaugh
Copy link
Member Author

@typescript-bot test this

@typescript-bot
Copy link
Collaborator

typescript-bot commented Aug 2, 2024

Starting jobs; this comment will be updated as builds start and complete.

Command Status Results
test top400 ✅ Started ✅ Results
user test this ✅ Started ✅ Results
run dt ✅ Started ✅ Results
perf test this faster ✅ Started 👀 Results

@typescript-bot
Copy link
Collaborator

Hey @RyanCavanaugh, the results of running the DT tests are ready.

Everything looks the same!

You can check the log here.

@typescript-bot
Copy link
Collaborator

@RyanCavanaugh Here are the results of running the user tests with tsc comparing main and refs/pull/56105/merge:

Everything looks good!

@typescript-bot
Copy link
Collaborator

@RyanCavanaugh
The results of the perf run you requested are in!

Here they are:

tsc

Comparison Report - baseline..pr
Metric baseline pr Delta Best Worst p-value
Compiler-Unions - node (v18.15.0, x64)
Errors 30 30 ~ ~ ~ p=1.000 n=6
Symbols 62,153 62,153 ~ ~ ~ p=1.000 n=6
Types 50,242 50,242 ~ ~ ~ p=1.000 n=6
Memory used 192,338k (± 0.09%) 194,090k (± 0.98%) +1,752k (+ 0.91%) 192,313k 195,856k p=0.030 n=6
Parse Time 1.95s (± 0.60%) 1.95s (± 0.54%) ~ 1.93s 1.96s p=0.619 n=6
Bind Time 1.06s (± 0.93%) 1.05s (± 0.79%) ~ 1.05s 1.07s p=0.588 n=6
Check Time 13.99s (± 0.44%) 13.97s (± 0.40%) ~ 13.89s 14.02s p=0.688 n=6
Emit Time 3.99s (± 1.31%) 3.99s (± 1.52%) ~ 3.88s 4.04s p=0.872 n=6
Total Time 20.99s (± 0.31%) 20.96s (± 0.38%) ~ 20.87s 21.07s p=0.373 n=6
angular-1 - node (v18.15.0, x64)
Errors 7 7 ~ ~ ~ p=1.000 n=6
Symbols 945,537 945,537 ~ ~ ~ p=1.000 n=6
Types 409,512 409,512 ~ ~ ~ p=1.000 n=6
Memory used 1,222,221k (± 0.01%) 1,222,270k (± 0.00%) ~ 1,222,211k 1,222,294k p=0.229 n=6
Parse Time 7.94s (± 1.00%) 7.91s (± 0.39%) ~ 7.88s 7.96s p=0.936 n=6
Bind Time 2.23s (± 0.60%) 2.23s (± 0.46%) ~ 2.21s 2.24s p=1.000 n=6
Check Time 36.22s (± 0.31%) 36.30s (± 0.29%) ~ 36.20s 36.40s p=0.226 n=6
Emit Time 17.83s (± 0.27%) 17.79s (± 0.31%) ~ 17.72s 17.88s p=0.199 n=6
Total Time 64.22s (± 0.25%) 64.23s (± 0.20%) ~ 64.04s 64.40s p=0.936 n=6
mui-docs - node (v18.15.0, x64)
Errors 0 0 ~ ~ ~ p=1.000 n=6
Symbols 2,483,478 2,483,478 ~ ~ ~ p=1.000 n=6
Types 1,012,731 1,012,731 ~ ~ ~ p=1.000 n=6
Memory used 2,475,470k (± 0.01%) 2,475,416k (± 0.00%) ~ 2,475,366k 2,475,462k p=0.471 n=6
Parse Time 10.39s (± 0.58%) 10.39s (± 0.56%) ~ 10.34s 10.49s p=1.000 n=6
Bind Time 2.60s (± 0.49%) 2.61s (± 0.51%) ~ 2.59s 2.62s p=0.383 n=6
Check Time 89.53s (± 1.65%) 89.61s (± 1.53%) ~ 88.13s 91.93s p=0.936 n=6
Emit Time 0.33s (± 1.58%) 0.33s (± 1.68%) ~ 0.32s 0.33s p=0.640 n=6
Total Time 102.85s (± 1.41%) 102.93s (± 1.38%) ~ 101.41s 105.30s p=0.936 n=6
self-build-src - node (v18.15.0, x64)
Errors 0 0 ~ ~ ~ p=1.000 n=6
Symbols 1,229,170 1,229,171 +1 (+ 0.00%) ~ ~ p=0.001 n=6
Types 264,285 264,285 ~ ~ ~ p=1.000 n=6
Memory used 2,526,692k (±11.74%) 2,527,270k (±11.74%) ~ 2,348,516k 3,059,422k p=0.575 n=6
Parse Time 7.58s (± 1.13%) 7.54s (± 1.15%) ~ 7.47s 7.71s p=0.261 n=6
Bind Time 2.78s (± 1.15%) 2.78s (± 1.16%) ~ 2.74s 2.82s p=0.872 n=6
Check Time 50.35s (± 0.70%) 50.43s (± 1.18%) ~ 49.72s 51.13s p=1.000 n=6
Emit Time 4.90s (± 1.73%) 4.92s (± 1.58%) ~ 4.83s 5.06s p=0.748 n=6
Total Time 65.63s (± 0.64%) 65.69s (± 1.00%) ~ 64.92s 66.45s p=0.936 n=6
self-build-src-public-api - node (v18.15.0, x64)
Errors 0 0 ~ ~ ~ p=1.000 n=6
Symbols 1,229,170 1,229,171 +1 (+ 0.00%) ~ ~ p=0.001 n=6
Types 264,285 264,285 ~ ~ ~ p=1.000 n=6
Memory used 2,422,454k (± 0.03%) 2,422,264k (± 0.01%) ~ 2,422,078k 2,422,564k p=0.810 n=6
Parse Time 6.22s (± 0.86%) 6.23s (± 0.84%) ~ 6.16s 6.31s p=0.872 n=6
Bind Time 2.03s (± 1.14%) 2.02s (± 0.54%) ~ 2.01s 2.04s p=0.374 n=6
Check Time 41.07s (± 0.31%) 41.15s (± 0.37%) ~ 41.02s 41.43s p=0.810 n=6
Emit Time 4.09s (± 2.06%) 4.17s (± 3.95%) ~ 4.02s 4.39s p=1.000 n=6
Total Time 53.43s (± 0.41%) 53.58s (± 0.36%) ~ 53.28s 53.80s p=0.230 n=6
self-compiler - node (v18.15.0, x64)
Errors 0 0 ~ ~ ~ p=1.000 n=6
Symbols 256,237 256,238 +1 (+ 0.00%) ~ ~ p=0.001 n=6
Types 104,976 104,976 ~ ~ ~ p=1.000 n=6
Memory used 428,238k (± 0.02%) 428,234k (± 0.02%) ~ 428,126k 428,312k p=0.810 n=6
Parse Time 4.17s (± 0.56%) 4.15s (± 0.36%) ~ 4.13s 4.17s p=0.506 n=6
Bind Time 1.61s (± 1.02%) 1.61s (± 2.09%) ~ 1.55s 1.64s p=0.373 n=6
Check Time 22.25s (± 0.28%) 22.20s (± 0.47%) ~ 22.08s 22.33s p=0.518 n=6
Emit Time 2.01s (± 1.12%) 2.03s (± 1.26%) ~ 2.00s 2.06s p=0.258 n=6
Total Time 30.04s (± 0.14%) 30.00s (± 0.38%) ~ 29.90s 30.18s p=0.423 n=6
ts-pre-modules - node (v18.15.0, x64)
Errors 35 35 ~ ~ ~ p=1.000 n=6
Symbols 224,931 224,931 ~ ~ ~ p=1.000 n=6
Types 94,146 94,146 ~ ~ ~ p=1.000 n=6
Memory used 370,155k (± 0.05%) 370,059k (± 0.03%) ~ 369,967k 370,269k p=0.810 n=6
Parse Time 3.46s (± 0.63%) 3.46s (± 0.89%) ~ 3.41s 3.49s p=0.871 n=6
Bind Time 1.91s (± 0.43%) 1.93s (± 0.61%) +0.02s (+ 0.78%) 1.91s 1.94s p=0.048 n=6
Check Time 19.32s (± 0.42%) 19.34s (± 0.29%) ~ 19.30s 19.45s p=0.936 n=6
Emit Time 0.00s 0.00s ~ ~ ~ p=1.000 n=6
Total Time 24.69s (± 0.35%) 24.73s (± 0.16%) ~ 24.67s 24.78s p=0.471 n=6
vscode - node (v18.15.0, x64)
Errors 0 0 ~ ~ ~ p=1.000 n=6
Symbols 2,990,977 2,990,977 ~ ~ ~ p=1.000 n=6
Types 1,029,155 1,029,155 ~ ~ ~ p=1.000 n=6
Memory used 3,116,199k (± 0.00%) 3,116,160k (± 0.00%) ~ 3,116,054k 3,116,250k p=0.810 n=6
Parse Time 13.78s (± 0.36%) 13.77s (± 0.33%) ~ 13.74s 13.86s p=0.374 n=6
Bind Time 4.29s (± 2.00%) 4.26s (± 0.41%) ~ 4.23s 4.28s p=0.868 n=6
Check Time 78.72s (± 0.31%) 78.57s (± 0.31%) ~ 78.28s 78.95s p=0.296 n=6
Emit Time 20.38s (± 0.29%) 20.43s (± 0.39%) ~ 20.34s 20.58s p=0.422 n=6
Total Time 117.18s (± 0.25%) 117.04s (± 0.16%) ~ 116.88s 117.36s p=0.298 n=6
webpack - node (v18.15.0, x64)
Errors 0 0 ~ ~ ~ p=1.000 n=6
Symbols 268,265 268,265 ~ ~ ~ p=1.000 n=6
Types 109,187 109,187 ~ ~ ~ p=1.000 n=6
Memory used 413,161k (± 0.04%) 413,076k (± 0.01%) ~ 412,997k 413,180k p=0.471 n=6
Parse Time 3.82s (± 0.27%) 3.82s (± 0.33%) ~ 3.81s 3.84s p=0.560 n=6
Bind Time 1.71s (± 0.80%) 1.71s (± 0.74%) ~ 1.70s 1.73s p=0.510 n=6
Check Time 16.92s (± 0.61%) 16.85s (± 0.36%) ~ 16.78s 16.95s p=0.198 n=6
Emit Time 0.00s 0.00s ~ ~ ~ p=1.000 n=6
Total Time 22.45s (± 0.44%) 22.38s (± 0.32%) ~ 22.31s 22.49s p=0.261 n=6
xstate-main - node (v18.15.0, x64)
Errors 0 0 ~ ~ ~ p=1.000 n=6
Symbols 538,543 538,543 ~ ~ ~ p=1.000 n=6
Types 176,469 176,469 ~ ~ ~ p=1.000 n=6
Memory used 481,365k (± 0.01%) 481,339k (± 0.01%) ~ 481,279k 481,408k p=0.471 n=6
Parse Time 3.41s (± 0.34%) 3.41s (± 0.43%) ~ 3.39s 3.43s p=0.680 n=6
Bind Time 1.26s (± 0.67%) 1.26s (± 0.41%) ~ 1.25s 1.26s p=0.533 n=6
Check Time 17.98s (± 0.27%) 18.04s (± 0.47%) ~ 17.96s 18.16s p=0.376 n=6
Emit Time 0.00s 0.00s ~ ~ ~ p=1.000 n=6
Total Time 22.64s (± 0.24%) 22.71s (± 0.34%) ~ 22.62s 22.80s p=0.370 n=6
System info unknown
Hosts
  • node (v18.15.0, x64)
Scenarios
  • Compiler-Unions - node (v18.15.0, x64)
  • angular-1 - node (v18.15.0, x64)
  • mui-docs - node (v18.15.0, x64)
  • self-build-src - node (v18.15.0, x64)
  • self-build-src-public-api - node (v18.15.0, x64)
  • self-compiler - node (v18.15.0, x64)
  • ts-pre-modules - node (v18.15.0, x64)
  • vscode - node (v18.15.0, x64)
  • webpack - node (v18.15.0, x64)
  • xstate-main - node (v18.15.0, x64)
Benchmark Name Iterations
Current pr 6
Baseline baseline 6

Developer Information:

Download Benchmarks

@typescript-bot
Copy link
Collaborator

@RyanCavanaugh Here are the results of running the top 400 repos with tsc comparing main and refs/pull/56105/merge:

Everything looks good!

@RyanCavanaugh RyanCavanaugh merged commit aafdfe5 into microsoft:main Aug 2, 2024
32 checks passed
@RyanCavanaugh RyanCavanaugh deleted the fix56094 branch August 2, 2024 22:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Author: Team For Backlog Bug PRs that fix a backlog bug
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Class members are accessible or inaccessible on this depending on how this is defined
4 participants