-
-
Notifications
You must be signed in to change notification settings - Fork 5.7k
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
[WIP] Add a benchmark for pull merging #4908
Conversation
73c6d12
to
94358d3
Compare
Codecov Report
@@ Coverage Diff @@
## master #4908 +/- ##
==========================================
- Coverage 37.56% 37.55% -0.01%
==========================================
Files 321 321
Lines 47206 47206
==========================================
- Hits 17732 17729 -3
+ Misses 26933 26931 -2
- Partials 2541 2546 +5
Continue to review full report at Codecov.
|
94358d3
to
aaa8583
Compare
e64d50c
to
08fcd16
Compare
@typeless is this ready to review? |
@lunny The repo tested in this PR is not big enough to confirm the effectiveness of #4921. I was considering what you suggested the other day using a Docker container for the prepared repo. But I have no a clear roadmap on how to do it (adding sane test fixtures to emulate a ready-for-merging-PRs repo is too hard). |
@typeless why close this? |
@lunny I think this PR is doing it the wrong way. It uses |
This is going to to be used for measuring the speed improvement of the pull merging code.