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

Fix RegExpIndicesArray by adding undefined to type definition #61079

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

stefnotch
Copy link

@stefnotch stefnotch commented Jan 29, 2025

Fixes #61078

@typescript-bot typescript-bot added the For Uncommitted Bug PR for untriaged, rejected, closed or missing bug label Jan 30, 2025
@RyanCavanaugh
Copy link
Member

@typescript-bot test it

@typescript-bot
Copy link
Collaborator

typescript-bot commented Feb 4, 2025

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/61079/merge:

Something interesting changed - please have a look.

Details

azure-sdk

/mnt/ts_downloads/_/m/azure-sdk/build.sh

  • [NEW] error TS2345: Argument of type '() => ts.AwaitExpression' is not assignable to parameter of type '(traversal: TransformTraversalControl) => Node'.
    • /mnt/ts_downloads/_/m/azure-sdk/src/util/admin/migrate-package/codemods/replaceChaiAsPromised.ts(48,26)

@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 34 34 ~ ~ ~ p=1.000 n=6
Symbols 62,390 62,390 ~ ~ ~ p=1.000 n=6
Types 50,395 50,395 ~ ~ ~ p=1.000 n=6
Memory used 193,127k (± 0.02%) 193,787k (± 0.76%) ~ 193,070k 196,758k p=1.000 n=6
Parse Time 1.31s (± 1.04%) 1.30s (± 0.63%) ~ 1.29s 1.31s p=0.111 n=6
Bind Time 0.73s 0.73s ~ ~ ~ p=1.000 n=6
Check Time 9.76s (± 0.31%) 9.74s (± 0.37%) ~ 9.71s 9.79s p=0.295 n=6
Emit Time 2.73s (± 0.54%) 2.72s (± 1.16%) ~ 2.66s 2.75s p=0.623 n=6
Total Time 14.53s (± 0.31%) 14.48s (± 0.44%) ~ 14.39s 14.56s p=0.421 n=6
angular-1 - node (v18.15.0, x64)
Errors 37 37 ~ ~ ~ p=1.000 n=6
Symbols 948,488 948,488 ~ ~ ~ p=1.000 n=6
Types 411,006 411,006 ~ ~ ~ p=1.000 n=6
Memory used 1,225,437k (± 0.00%) 1,225,439k (± 0.00%) ~ 1,225,383k 1,225,509k p=1.000 n=6
Parse Time 6.65s (± 0.86%) 6.65s (± 0.85%) ~ 6.56s 6.71s p=1.000 n=6
Bind Time 1.89s (± 0.78%) 1.89s (± 0.40%) ~ 1.88s 1.90s p=0.620 n=6
Check Time 31.91s (± 0.18%) 32.04s (± 0.21%) +0.13s (+ 0.42%) 31.93s 32.12s p=0.013 n=6
Emit Time 15.17s (± 0.54%) 15.21s (± 0.30%) ~ 15.12s 15.24s p=0.173 n=6
Total Time 55.61s (± 0.28%) 55.79s (± 0.16%) ~ 55.66s 55.87s p=0.065 n=6
mui-docs - node (v18.15.0, x64)
Errors 0 0 ~ ~ ~ p=1.000 n=6
Symbols 2,447,815 2,447,815 ~ ~ ~ p=1.000 n=6
Types 898,759 898,759 ~ ~ ~ p=1.000 n=6
Memory used 2,310,423k (± 0.00%) 2,310,426k (± 0.00%) ~ 2,310,386k 2,310,474k p=1.000 n=6
Parse Time 8.93s (± 0.20%) 8.93s (± 0.36%) ~ 8.90s 8.98s p=0.808 n=6
Bind Time 2.16s (± 0.29%) 2.17s (± 0.57%) ~ 2.14s 2.17s p=0.116 n=6
Check Time 74.42s (± 0.38%) 74.20s (± 0.39%) ~ 73.86s 74.62s p=0.298 n=6
Emit Time 0.28s (± 2.67%) 0.28s (± 2.70%) ~ 0.27s 0.29s p=0.487 n=6
Total Time 85.79s (± 0.31%) 85.57s (± 0.33%) ~ 85.29s 85.99s p=0.298 n=6
self-build-src - node (v18.15.0, x64)
Errors 0 0 ~ ~ ~ p=1.000 n=6
Symbols 1,229,397 1,229,397 ~ ~ ~ p=1.000 n=6
Types 267,085 267,085 ~ ~ ~ p=1.000 n=6
Memory used 2,362,143k (± 0.02%) 2,361,769k (± 0.03%) ~ 2,360,791k 2,362,686k p=0.471 n=6
Parse Time 5.21s (± 0.94%) 5.19s (± 0.89%) ~ 5.13s 5.27s p=0.521 n=6
Bind Time 1.79s (± 0.35%) 1.79s (± 0.99%) ~ 1.75s 1.80s p=1.000 n=6
Check Time 35.25s (± 0.28%) 35.24s (± 0.20%) ~ 35.16s 35.33s p=0.810 n=6
Emit Time 3.01s (± 1.27%) 2.99s (± 1.00%) ~ 2.97s 3.05s p=0.334 n=6
Total Time 45.27s (± 0.33%) 45.22s (± 0.19%) ~ 45.14s 45.37s p=0.630 n=6
self-build-src-public-api - node (v18.15.0, x64)
Errors 0 0 ~ ~ ~ p=1.000 n=6
Symbols 1,229,397 1,229,397 ~ ~ ~ p=1.000 n=6
Types 267,085 267,085 ~ ~ ~ p=1.000 n=6
Memory used 3,038,432k (± 9.75%) 3,039,742k (± 9.73%) ~ 2,435,401k 3,161,527k p=0.173 n=6
Parse Time 7.02s (± 1.39%) 7.00s (± 1.05%) ~ 6.91s 7.10s p=0.748 n=6
Bind Time 2.15s (± 2.11%) 2.15s (± 0.63%) ~ 2.14s 2.18s p=0.871 n=6
Check Time 42.93s (± 0.57%) 43.02s (± 0.43%) ~ 42.82s 43.29s p=0.810 n=6
Emit Time 3.51s (± 1.66%) 3.53s (± 2.76%) ~ 3.43s 3.69s p=0.936 n=6
Total Time 55.60s (± 0.58%) 55.69s (± 0.46%) ~ 55.38s 56.08s p=0.936 n=6
self-compiler - node (v18.15.0, x64)
Errors 0 0 ~ ~ ~ p=1.000 n=6
Symbols 263,074 263,074 ~ ~ ~ p=1.000 n=6
Types 106,881 106,881 ~ ~ ~ p=1.000 n=6
Memory used 441,322k (± 0.01%) 441,298k (± 0.02%) ~ 441,182k 441,389k p=0.810 n=6
Parse Time 3.55s (± 1.39%) 3.55s (± 1.09%) ~ 3.50s 3.60s p=0.627 n=6
Bind Time 1.32s (± 1.00%) 1.31s (± 0.57%) ~ 1.30s 1.32s p=0.094 n=6
Check Time 19.07s (± 0.55%) 19.03s (± 0.30%) ~ 18.92s 19.07s p=0.573 n=6
Emit Time 1.55s (± 1.46%) 1.53s (± 0.99%) ~ 1.50s 1.54s p=0.167 n=6
Total Time 25.48s (± 0.46%) 25.42s (± 0.27%) ~ 25.30s 25.50s p=0.336 n=6
ts-pre-modules - node (v18.15.0, x64)
Errors 70 70 ~ ~ ~ p=1.000 n=6
Symbols 226,113 226,113 ~ ~ ~ p=1.000 n=6
Types 94,488 94,488 ~ ~ ~ p=1.000 n=6
Memory used 371,726k (± 0.01%) 371,804k (± 0.02%) +78k (+ 0.02%) 371,725k 371,917k p=0.045 n=6
Parse Time 2.88s (± 1.47%) 2.88s (± 0.89%) ~ 2.86s 2.93s p=0.809 n=6
Bind Time 1.60s (± 1.48%) 1.60s (± 1.44%) ~ 1.57s 1.63s p=1.000 n=6
Check Time 16.42s (± 0.31%) 16.41s (± 0.22%) ~ 16.35s 16.46s p=0.567 n=6
Emit Time 0.00s 0.00s ~ ~ ~ p=1.000 n=6
Total Time 20.90s (± 0.50%) 20.89s (± 0.21%) ~ 20.85s 20.96s p=0.936 n=6
vscode - node (v18.15.0, x64)
Errors 3 5 🔻+2 (+66.67%) ~ ~ p=0.001 n=6
Symbols 3,274,571 3,274,571 ~ ~ ~ p=1.000 n=6
Types 1,126,273 1,126,275 +2 (+ 0.00%) ~ ~ p=0.001 n=6
Memory used 3,349,300k (± 0.01%) 3,349,501k (± 0.01%) ~ 3,348,874k 3,349,731k p=0.298 n=6
Parse Time 14.56s (± 0.46%) 14.54s (± 0.43%) ~ 14.44s 14.61s p=0.748 n=6
Bind Time 4.60s (± 0.83%) 4.58s (± 0.57%) ~ 4.56s 4.63s p=0.224 n=6
Check Time 89.50s (± 2.94%) 88.68s (± 1.29%) ~ 87.80s 90.97s p=0.936 n=6
Emit Time 28.22s (± 2.93%) 27.63s (± 2.37%) ~ 27.24s 28.96s p=0.109 n=6
Total Time 136.88s (± 1.91%) 135.44s (± 0.86%) ~ 134.29s 137.36s p=0.575 n=6
webpack - node (v18.15.0, x64)
Errors 0 0 ~ ~ ~ p=1.000 n=6
Symbols 292,529 292,529 ~ ~ ~ p=1.000 n=6
Types 119,113 119,113 ~ ~ ~ p=1.000 n=6
Memory used 446,144k (± 0.03%) 446,054k (± 0.02%) ~ 445,934k 446,135k p=0.230 n=6
Parse Time 4.07s (± 0.94%) 4.07s (± 1.09%) ~ 4.01s 4.13s p=1.000 n=6
Bind Time 1.78s (± 0.79%) 1.78s (± 0.50%) ~ 1.77s 1.79s p=1.000 n=6
Check Time 18.86s (± 0.45%) 18.79s (± 0.66%) ~ 18.57s 18.95s p=0.228 n=6
Emit Time 0.00s 0.00s ~ ~ ~ p=1.000 n=6
Total Time 24.71s (± 0.44%) 24.64s (± 0.35%) ~ 24.50s 24.76s p=0.148 n=6
xstate-main - node (v18.15.0, x64)
Errors 5 5 ~ ~ ~ p=1.000 n=6
Symbols 555,375 555,375 ~ ~ ~ p=1.000 n=6
Types 186,146 186,146 ~ ~ ~ p=1.000 n=6
Memory used 494,570k (± 0.01%) 494,530k (± 0.01%) ~ 494,473k 494,579k p=0.378 n=6
Parse Time 4.25s (± 0.54%) 4.24s (± 0.57%) ~ 4.20s 4.27s p=0.416 n=6
Bind Time 1.47s (± 0.71%) 1.48s (± 0.99%) ~ 1.47s 1.50s p=0.557 n=6
Check Time 24.30s (± 0.35%) 24.31s (± 0.77%) ~ 24.12s 24.67s p=0.575 n=6
Emit Time 0.00s 0.00s ~ ~ ~ p=1.000 n=6
Total Time 30.03s (± 0.27%) 30.03s (± 0.60%) ~ 29.86s 30.37s p=0.630 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/61079/merge:

Something interesting changed - please have a look.

Details

microsoft/vscode

5 of 55 projects failed to build with the old tsc and were ignored

src/tsconfig.tsec.json

src/tsconfig.monaco.json

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
For Uncommitted Bug PR for untriaged, rejected, closed or missing bug
Projects
Status: Not started
Development

Successfully merging this pull request may close these issues.

RegExpIndicesArray type definition incomplete
3 participants