rust-analyzer / expect-test

Minimalistic snapshot testing for Rust.
Apache License 2.0
248 stars 22 forks source link

Use dissimilar for diffs instead of difference #11

Closed Jesse-Bakker closed 3 years ago

Jesse-Bakker commented 3 years ago

As discussed with @matklad in https://github.com/rust-analyzer/rust-analyzer/pull/7106.

Before: expect-test-screen-difference

After: expect-test-screen

Used background instead of foreground colours because I think the diff boundaries are clearer, but I'm not married to that change ;)

matklad commented 3 years ago

Perfect, thanks!

Please bump Cargo.toml's version to 1.1.0, so that this gets released by CI.

bors d+

bors[bot] commented 3 years ago

:v: Jesse-Bakker can now approve this pull request. To approve and merge a pull request, simply reply with bors r+. More detailed instructions are available here.

Jesse-Bakker commented 3 years ago

bors r+

bors[bot] commented 3 years ago

Build succeeded!

And happy new year! 🎉