Skip to content

chore: use oxlint --type-check#345

Merged
mizdra merged 1 commit intomainfrom
type-check-in-linting
Feb 21, 2026
Merged

chore: use oxlint --type-check#345
mizdra merged 1 commit intomainfrom
type-check-in-linting

Conversation

@mizdra
Copy link
Copy Markdown
Owner

@mizdra mizdra commented Feb 21, 2026

The lint script is now 1.17 to 1.96 times faster and simpler.

npm run lint (without incremental build) vs npm run lint

$ hyperfine --prepare="git switch main && npx tsc -b --clean" "echo main && npm run lint" --prepare="git switch type-check-in-linting" "echo type-check-in-linting && npm run lint"
Benchmark 1: echo main && npm run lint
  Time (mean ± σ):      2.224 s ±  0.009 s    [User: 4.727 s, System: 0.630 s]
  Range (min … max):    2.210 s …  2.236 s    10 runs

Benchmark 2: echo type-check-in-linting && npm run lint
  Time (mean ± σ):      1.132 s ±  0.007 s    [User: 2.681 s, System: 0.505 s]
  Range (min … max):    1.121 s …  1.142 s    10 runs

Summary
  echo type-check-in-linting && npm run lint ran
    1.96 ± 0.01 times faster than echo main && npm run lint

npm run lint (with incremental build) vs npm run lint

$ hyperfine --prepare="git switch main" "echo main && npm run lint" --prepare="git switch type-check-in-linting" "echo type-check-in-linting && npm run lint"
Benchmark 1: echo main && npm run lint
  Time (mean ± σ):      1.344 s ±  0.041 s    [User: 2.755 s, System: 0.532 s]
  Range (min … max):    1.311 s …  1.434 s    10 runs

Benchmark 2: echo type-check-in-linting && npm run lint
  Time (mean ± σ):      1.147 s ±  0.036 s    [User: 2.690 s, System: 0.491 s]
  Range (min … max):    1.107 s …  1.222 s    10 runs

Summary
  echo type-check-in-linting && npm run lint ran
    1.17 ± 0.05 times faster than echo main && npm run lint

@changeset-bot
Copy link
Copy Markdown

changeset-bot bot commented Feb 21, 2026

⚠️ No Changeset found

Latest commit: 0d69a5c

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

packages/*/tsconfig.build.tsbuildinfo
tsconfig.tsbuildinfo
key: lint-tools-${{ runner.arch }}-${{ runner.os }}-${{ github.sha }}
restore-keys: lint-tools-${{ runner.arch }}-${{ runner.os }}
Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

Oxfmt and Oxlint do not have a cache mechanism. oxc-project/tsgolint does not support increment builds. Therefore, there are no files that should be cached in actions/cache.

@mizdra mizdra marked this pull request as ready for review February 21, 2026 08:16
@mizdra mizdra merged commit ba30662 into main Feb 21, 2026
19 checks passed
@mizdra mizdra deleted the type-check-in-linting branch February 21, 2026 08:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant