Closed nflaig closed 1 week ago
Attention: Patch coverage is 64.28571%
with 5 lines
in your changes missing coverage. Please review.
Project coverage is 62.75%. Comparing base (
85dc0ba
) to head (382b188
).
✔️ no performance regression detected
🚀🚀 Significant benchmark improvement detected
Benchmark suite | Current: d441a0f73a86ac47e93a81bbea583868cdef26b0 | Previous: 85dc0ba06d1adaf05df0e4aac8f74dee267c9625 | Ratio |
---|---|---|---|
getExpectedWithdrawals 250000 eb:0.95,eth1:0.7,we:0.05,wn:0,smpl:18 | 3.2400 us/op | 13.027 us/op | 0.25 |
by benchmarkbot/action
Motivation
Noticed Grandine does not return any error response, not even a text/plain response. To improve UX for such cases, we can default to http status text to avoid empty error messages.
Description
Default to http status text if error response is empty