-
Notifications
You must be signed in to change notification settings - Fork 499
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
chore: add codspeed for benchmark #5280
Conversation
Hi, @dqhl76, token has been created at |
CodSpeed Performance ReportCongrats! CodSpeed is installed 🎉
You will start to see performance impacts in the reports once the benchmarks are run from your default branch.
|
The benchmark is identified by name, and we can expand it to support various services. We can implement this in future PRs. |
Hi @Xuanwo, All comments are applied, could you please take a look again? Thanks! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you, @dqhl76, for working on this. Also, thank you, @adriencaccia, for building this great service and helping us with the integration.
Which issue does this PR close?
Closes #4937 .
Rationale for this change
What changes are included in this PR?
A new workflow that runs on every PR and the default main branch.
The criterion dependency is changed to codspeed-criterion-compat.
This should not affect cargo bench
Are there any user-facing changes?