Bug in assert_matches deprecation message

[smoelius]

Currently, it says:

snapbox/crates/snapbox/src/lib.rs

Line 200 in 60703c9

#[deprecated(since = "0.5.11", note = "Replaced with `assert_data_eq!(actual)`")]

I think it should say something like the following?

Replaced with `assert_data_eq!(expected, actual)`
                               ^^^^^^^^^^

[smoelius]

I know it's a little bikesheddy, but can I ask why you went with the actual, expected order?

[epage]

I personally prefer (actual, expected) asserts (and if conditions) but our output is of the model diff expected actual which suggests (expected, actual), so I think that is why (expected, actual) was used. This came up again in #226 and that was enough to push me over the edge to switch them but I still go back and forth.

[smoelius]

Thank you vey much for your reply.