Skip to content

feat: refactored benchmarks and added CliBenchmarks - #1896

Open
TimothyMakkison wants to merge 1 commit into
belav:mainfrom
TimothyMakkison:cli_bench
Open

feat: refactored benchmarks and added CliBenchmarks#1896
TimothyMakkison wants to merge 1 commit into
belav:mainfrom
TimothyMakkison:cli_bench

Conversation

@TimothyMakkison

@TimothyMakkison TimothyMakkison commented Aug 6, 2026

Copy link
Copy Markdown
Contributor
  • Added CliBenchmarks containing benchmarks for formatting a directory and checking a list of files.
  • Benchmarks is getting cluttered so I split it up into CSharpBenchmarks and XmlBenchmarks

Relevant to #1893

@TimothyMakkison
TimothyMakkison marked this pull request as ready for review August 8, 2026 17:02
@TimothyMakkison

Copy link
Copy Markdown
Contributor Author

Should work, but I'm a little confused.

When I run dotnet csharpier format . at the root of my csharpier repo it takes 3 seconds
Formatted 447 files in 3881ms.

FormatCli should be identical and yet it runs faster than in terminal
Formatted 447 files in 872ms.

I assume this discrepancy is because the method is allowed to warmup beforehand 🤔

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