shotover / shotover-proxy

L7 data-layer proxy
https://docs.shotover.io
Apache License 2.0
83 stars 16 forks source link

pretty assert #1624

Closed conorbros closed 3 months ago

conorbros commented 3 months ago

Seeing if there is interest in using this library. An example can be seen in the logs.

rukai commented 3 months ago

Woah, I love this! Lets set it to a dev-dependency instead of regular dependency and land this!

For the review who sees this after the intentional failure is removed, the assert_eq now has colored highlighting and looks like this: image

codspeed-hq[bot] commented 3 months ago

CodSpeed Performance Report

Merging #1624 will improve performances by 10.94%

Comparing conorbros:pretty-assert (1c6244b) with main (c1331f8)

Summary

⚡ 1 improvements ✅ 36 untouched benchmarks

Benchmarks breakdown

Benchmark main conorbros:pretty-assert Change
encode_system.local_result_v5_no_compression 491.1 µs 442.7 µs +10.94%
rukai commented 3 months ago

Is the test failure intentional?