Skip to content

chore: migrate to tsgo and typescript 6.x [AR-51963]#216

Open
StyleShit wants to merge 23 commits intodrivenets:mainfrom
StyleShit:chore/AR-51936-tsgo-ts-6
Open

chore: migrate to tsgo and typescript 6.x [AR-51963]#216
StyleShit wants to merge 23 commits intodrivenets:mainfrom
StyleShit:chore/AR-51936-tsgo-ts-6

Conversation

@StyleShit
Copy link
Copy Markdown
Member

@StyleShit StyleShit commented Mar 5, 2026 •
edited
Loading

tsgo is about x4.3 faster than tsc when running in the design system package:

hyperfine "pnpm tsc" "pnpm tsgo"
Benchmark 1: pnpm tsc
  Time (mean ± σ):      4.233 s ±  0.094 s    [User: 6.848 s, System: 0.536 s]
  Range (min … max):    4.121 s …  4.406 s    10 runs
 
Benchmark 2: pnpm tsgo
  Time (mean ± σ):     986.7 ms ±  34.2 ms    [User: 3397.4 ms, System: 502.7 ms]
  Range (min … max):   951.3 ms … 1044.2 ms    10 runs
 
Summary
  pnpm tsgo ran
    4.29 ± 0.18 times faster than pnpm tsc

StyleShit
StyleShit commented Mar 8, 2026
View reviewed changes
Comment thread tsconfig.base.json
Comment on lines -15 to -19
"strict": true,
"jsx": "react-jsx",
"verbatimModuleSyntax": true,
"isolatedModules": true,
"noUncheckedSideEffectImports": true,
Copy link
Copy Markdown
Member Author

@StyleShit StyleShit Mar 8, 2026

Choose a reason for hiding this comment

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

these are the new defaults

mmurawski-dn
mmurawski-dn previously approved these changes Apr 14, 2026
View reviewed changes
@StyleShit StyleShit marked this pull request as ready for review April 15, 2026 05:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mmurawski-dn mmurawski-dn mmurawski-dn left review comments

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

commitlint-plugin-internal cursor PR that changes cursor rules, skills, etc. dependencies design-system eslint-plugin eslint-plugin-internal tooling vite-plugin

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@StyleShit @mmurawski-dn