Skip to content

chore: add a sample benchmark#741

Merged
graphite-app[bot] merged 1 commit intomainfrom
02-24-chore_add_a_sample_benchmark
Feb 25, 2026
Merged

chore: add a sample benchmark#741
graphite-app[bot] merged 1 commit intomainfrom
02-24-chore_add_a_sample_benchmark

Conversation

@camchenry
Copy link
Copy Markdown
Member

@camchenry camchenry commented Feb 25, 2026

In an effort to get more serious about benchmarking tsgolint's performance, I'd like to add this simple benchmark that just measures the time it takes to run the linter alone (without program creation and I/O). Although we aren't benchmarking in CI, this should make it easier to compare performance between branches running locally.

It's not a perfect benchmark, but we can add more in the future to account for things like the time it takes to create the programs and streaming diagnostics back to oxlint.

Copy link
Copy Markdown
Member Author

camchenry commented Feb 25, 2026


How to use the Graphite Merge Queue

Add the label 0-merge to this PR to add it to the merge queue.

You must have a Graphite account in order to use the merge queue. Sign up using this link.

An organization admin has enabled the Graphite Merge Queue in this repository.

Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue.

This stack of pull requests is managed by Graphite. Learn more about stacking.

@camchenry camchenry marked this pull request as ready for review February 25, 2026 02:42
@camchenry camchenry force-pushed the 02-24-chore_add_a_sample_benchmark branch from fbc7ff1 to 0947a77 Compare February 25, 2026 02:54
Copy link
Copy Markdown
Member Author

camchenry commented Feb 25, 2026

Merge activity

In an effort to get more serious about benchmarking tsgolint's performance, I'd like to add this simple benchmark that just measures the time it takes to run the linter alone (without program creation and I/O). Although we aren't benchmarking in CI, this should make it easier to compare performance between branches running locally.

It's not a perfect benchmark, but we can add more in the future to account for things like the time it takes to create the programs and streaming diagnostics back to oxlint.
@camchenry
Copy link
Copy Markdown
Member Author

Going to merge since this is mainly just a development thing - shouldn't impact any actual running code or tests

@graphite-app graphite-app Bot force-pushed the 02-24-chore_add_a_sample_benchmark branch from 0947a77 to 865254c Compare February 25, 2026 03:36
@graphite-app graphite-app Bot merged commit 865254c into main Feb 25, 2026
8 checks passed
@graphite-app graphite-app Bot deleted the 02-24-chore_add_a_sample_benchmark branch February 25, 2026 03:43
@graphite-app graphite-app Bot removed the 0-merge label Feb 25, 2026
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