Skip to content

Add benchmark for testing all rules individually#1469

Merged
anderseknert merged 1 commit intomainfrom
bench-lint-all-rules
Apr 8, 2025
Merged

Add benchmark for testing all rules individually#1469
anderseknert merged 1 commit intomainfrom
bench-lint-all-rules

Conversation

@anderseknert
Copy link
Member

This is currently very expensive/slow, but it's still a good benchmark to keep around for knowing the cost of some rule relative to others.

@anderseknert anderseknert force-pushed the bench-lint-all-rules branch from 6fccc97 to 961cff8 Compare April 7, 2025 20:18
Copy link
Contributor

@charlieegan3 charlieegan3 left a comment

Choose a reason for hiding this comment

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

Sounds good to me!

This is currently very expensive/slow, but it's still a good benchmark
to keep around for knowing the cost of some rule relative to others.

Signed-off-by: Anders Eknert <anders@styra.com>
@anderseknert anderseknert force-pushed the bench-lint-all-rules branch from 961cff8 to 754fe86 Compare April 8, 2025 10:01
@anderseknert anderseknert merged commit 9e6f039 into main Apr 8, 2025
5 checks passed
@anderseknert anderseknert deleted the bench-lint-all-rules branch April 8, 2025 10:09
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