Skip to content

Consistently check assignability to template literal placeholders - #56598

Merged
Anders Hejlsberg (ahejlsberg) merged 2 commits into
mainfrom
fix56582
Nov 29, 2023
Merged

Consistently check assignability to template literal placeholders#56598
Anders Hejlsberg (ahejlsberg) merged 2 commits into
mainfrom
fix56582

Conversation

@ahejlsberg

Copy link
Copy Markdown
Member

Fixes#56582.

@ahejlsberg

Copy link
Copy Markdown
MemberAuthor

@typescript-bot

TypeScript Bot (typescript-bot) commented Nov 29, 2023

Copy link
Copy Markdown
Contributor

Heya Anders Hejlsberg (@ahejlsberg), I've started to run the parallelized Definitely Typed test suite on this PR at 0e4c652. You can monitor the build here.

Update: The results are in!

@typescript-bot

TypeScript Bot (typescript-bot) commented Nov 29, 2023

Copy link
Copy Markdown
Contributor

Heya Anders Hejlsberg (@ahejlsberg), I've started to run the diff-based user code test suite on this PR at 0e4c652. You can monitor the build here.

Update: The results are in!

@typescript-bot

TypeScript Bot (typescript-bot) commented Nov 29, 2023

Copy link
Copy Markdown
Contributor

Heya Anders Hejlsberg (@ahejlsberg), I've started to run the tsc-only perf test suite on this PR at 0e4c652. You can monitor the build here.

Update: The results are in!

@typescript-bot

TypeScript Bot (typescript-bot) commented Nov 29, 2023

Copy link
Copy Markdown
Contributor

Heya Anders Hejlsberg (@ahejlsberg), I've started to run the diff-based top-repos suite on this PR at 0e4c652. You can monitor the build here.

Update: The results are in!

@typescript-bot

Copy link
Copy Markdown
Contributor

Anders Hejlsberg (@ahejlsberg)
The results of the perf run you requested are in!

Here they are:

Compiler

Comparison Report - baseline..pr
MetricbaselineprDeltaBestWorstp-value
Angular - node (v18.15.0, x64)
Memory used295,211k (± 0.01%)295,205k (± 0.01%)~295,173k295,246kp=0.688 n=6
Parse Time2.64s (± 0.20%)2.64s (± 0.20%)~2.64s2.65sp=1.000 n=6
Bind Time0.82s (± 0.00%)0.82s (± 0.00%)~0.82s0.82sp=1.000 n=6
Check Time8.04s (± 0.22%)8.04s (± 0.25%)~8.02s8.08sp=0.743 n=6
Emit Time7.08s (± 0.30%)7.08s (± 0.46%)~7.05s7.14sp=0.745 n=6
Total Time18.59s (± 0.09%)18.58s (± 0.19%)~18.53s18.62sp=0.870 n=6
Compiler-Unions - node (v18.15.0, x64)
Memory used190,671k (± 0.01%)195,608k (± 1.24%)+4,938k (+ 2.59%)190,641k196,691kp=0.045 n=6
Parse Time1.37s (± 0.85%)1.36s (± 1.20%)~1.34s1.38sp=0.680 n=6
Bind Time0.72s (± 0.00%)0.72s (± 0.00%)~0.72s0.72sp=1.000 n=6
Check Time9.15s (± 0.49%)9.17s (± 0.48%)~9.10s9.23sp=0.374 n=6
Emit Time2.62s (± 0.32%)2.61s (± 0.51%)~2.59s2.63sp=0.612 n=6
Total Time13.85s (± 0.33%)13.86s (± 0.34%)~13.81s13.93sp=0.746 n=6
Monaco - node (v18.15.0, x64)
Memory used347,357k (± 0.00%)347,345k (± 0.00%)~347,322k347,364kp=0.230 n=6
Parse Time2.46s (± 0.60%)2.46s (± 0.36%)~2.45s2.47sp=0.804 n=6
Bind Time0.93s (± 0.56%)0.93s (± 0.56%)~0.92s0.93sp=1.000 n=6
Check Time6.89s (± 0.36%)6.89s (± 0.35%)~6.86s6.93sp=0.746 n=6
Emit Time4.05s (± 0.43%)4.05s (± 0.40%)~4.04s4.08sp=1.000 n=6
Total Time14.32s (± 0.27%)14.33s (± 0.26%)~14.29s14.40sp=0.686 n=6
TFS - node (v18.15.0, x64)
Memory used302,618k (± 0.00%)302,620k (± 0.01%)~302,588k302,649kp=0.810 n=6
Parse Time2.00s (± 0.27%)2.00s (± 1.03%)~1.98s2.03sp=1.000 n=6
Bind Time1.00s (± 0.52%)1.00s (± 0.98%)~0.99s1.02sp=0.386 n=6
Check Time6.26s (± 0.37%)6.25s (± 0.26%)~6.23s6.28sp=0.255 n=6
Emit Time3.57s (± 0.46%)3.57s (± 0.37%)~3.55s3.59sp=0.621 n=6
Total Time12.83s (± 0.22%)12.83s (± 0.20%)~12.80s12.86sp=0.872 n=6
material-ui - node (v18.15.0, x64)
Memory used470,598k (± 0.01%)470,664k (± 0.02%)~470,588k470,804kp=0.199 n=6
Parse Time2.58s (± 0.53%)2.58s (± 0.32%)~2.57s2.59sp=0.197 n=6
Bind Time0.98s (± 0.56%)0.99s (± 0.83%)~0.98s1.00sp=0.859 n=6
Check Time16.76s (± 0.24%)16.74s (± 0.39%)~16.64s16.83sp=0.628 n=6
Emit Time0.00s (± 0.00%)0.00s (± 0.00%)~0.00s0.00sp=1.000 n=6
Total Time20.33s (± 0.19%)20.31s (± 0.34%)~20.20s20.39sp=0.574 n=6
xstate - node (v18.15.0, x64)
Memory used512,020k (± 0.01%)512,030k (± 0.01%)~511,969k512,152kp=0.936 n=6
Parse Time3.28s (± 0.26%)3.28s (± 0.16%)~3.27s3.28sp=0.923 n=6
Bind Time1.54s (± 0.26%)1.53s (± 0.76%)~1.51s1.54sp=0.056 n=6
Check Time2.80s (± 0.66%)2.81s (± 0.59%)~2.79s2.83sp=0.371 n=6
Emit Time0.08s (± 4.99%)0.08s (± 0.00%)~0.08s0.08sp=0.405 n=6
Total Time7.69s (± 0.19%)7.70s (± 0.32%)~7.67s7.72sp=1.000 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)
BenchmarkNameIterations
Currentpr6
Baselinebaseline6

Developer Information:

Download Benchmarks

@typescript-bot

Copy link
Copy Markdown
Contributor

Anders Hejlsberg (@ahejlsberg) Here are the results of running the user test suite comparing main and refs/pull/56598/merge:

There were infrastructure failures potentially unrelated to your change:

  • 2 instances of "Package install failed"

Otherwise...

Something interesting changed - please have a look.

Details

puppeteer

packages/browsers/test/src/tsconfig.json

@typescript-bot

Copy link
Copy Markdown
Contributor

Hey Anders Hejlsberg (@ahejlsberg), the results of running the DT tests are ready.
Everything looks the same!
You can check the log here.

@typescript-bot

Copy link
Copy Markdown
Contributor

Anders Hejlsberg (@ahejlsberg) Here are the results of running the top-repos suite comparing main and refs/pull/56598/merge:

Everything looks good!

@microsoftMicrosoft (microsoft) locked as resolved and limited conversation to collaborators Oct 16, 2025
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Author: TeamFor Milestone BugPRs that fix a bug with a specific milestone

Projects

None yet

Development

Successfully merging this pull request may close these issues.

String templates doesn't work with nested generics and keyof

3 participants

@ahejlsberg@typescript-bot@weswigham