Skip to content

Conversation

@TimothyMakkison
Copy link
Contributor

Add benchmarks for all source generators

DynamicTestsGeneratorBenchmarks

Method Mean Error StdDev Gen0 Gen1 Allocated
RunGenerator 2.351 ms 0.0219 ms 0.0313 ms 42.9688 11.7188 407.67 KB

InfrastructureGeneratorBenchmarks

Method Mean Error StdDev Gen0 Gen1 Allocated
RunGenerator 244.6 us 3.40 us 3.01 us 24.4141 2.9297 231.92 KB

HookMetadataGeneratorBenchmarks

Method Mean Error StdDev Gen0 Gen1 Allocated
RunGenerator 43.19 ms 0.862 ms 1.316 ms 4000.0000 1333.3333 36.42 MB

PropertyInjectionSourceGeneratorBenchmarks

Method Mean Error StdDev Median Gen0 Allocated
RunGenerator 72.91 ms 1.955 ms 5.418 ms 70.90 ms 1000.0000 10.42 MB

@claude
Copy link
Contributor

claude bot commented Feb 3, 2026

Code review

No issues found. Checked for bugs and CLAUDE.md compliance.

The PR adds comprehensive benchmark coverage for all source generators following the established patterns in the codebase. The changes are consistent, well-structured, and the performance metrics documented in the PR description provide valuable baseline data for future optimization work.

@thomhurst thomhurst enabled auto-merge (squash) February 3, 2026 23:04
@thomhurst thomhurst merged commit a49c379 into thomhurst:main Feb 3, 2026
8 of 10 checks passed
This was referenced Feb 5, 2026
This was referenced Feb 9, 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.

2 participants