-
-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Add codspeed benchmarks #9534
Add codspeed benchmarks #9534
Conversation
This will get the ball rolling so we know when we have solved #9520
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.
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
✅ All tests successful. No failed tests found. Additional details and impacted files@@ Coverage Diff @@
## master #9534 +/- ##
==========================================
+ Coverage 97.83% 98.59% +0.76%
==========================================
Files 103 107 +4
Lines 35035 35131 +96
Branches 4152 4180 +28
==========================================
+ Hits 34277 34639 +362
+ Misses 558 329 -229
+ Partials 200 163 -37
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Backport to 3.10: 💔 cherry-picking failed — conflicts found❌ Failed to cleanly apply 5a79c0f on top of patchback/backports/3.10/5a79c0f361129f6276a892bf81eed92f22b729d8/pr-9534 Backporting merged PR #9534 into master
🤖 @patchback |
Backport to 3.11: 💚 backport PR created✅ Backport PR branch: Backported as #9536 🤖 @patchback |
(cherry picked from commit 5a79c0f)
(cherry picked from commit 5a79c0f)
What do these changes do?
Adds codspeed benchmarks
This will get the ball rolling so we know when we have solved #9520
Are there changes in behavior for the user?
no
Is it a substantial burden for the maintainers to support this?
no