-
Notifications
You must be signed in to change notification settings - Fork 2.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
feat: change the way parameters are returned from Executors #4243
Conversation
… parameters but merge them
…ults when merging params
fix: use single quote
Co-authored-by: Tobias Jacobowitz <tobias.jacobowitz@posteo.de>
Latency summaryCurrent PR yields:
Breakdown
Backed by latency-tracking. Further commits will update this comment. |
the test named |
Codecov Report
@@ Coverage Diff @@
## master #4243 +/- ##
==========================================
+ Coverage 87.16% 87.23% +0.06%
==========================================
Files 113 113
Lines 8254 8286 +32
==========================================
+ Hits 7195 7228 +33
+ Misses 1059 1058 -1
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
follow up on this PR which is somehow broken on the github side