search

mitsuhiko / insta

A snapshot testing library for rust
https://insta.rs
Apache License 2.0
2.26k stars 102 forks source link

Add warning about stability to doc comment for internals module #648

Closed lasernoises closed 1 month ago

lasernoises commented 1 month ago

As discussed in #610 it probably makes sense to be explicit about the internals module not being guaranteed to be stable.

I also noticed that TextSnapshotKind gets exposed globally. This was added in #581 when it was still called SnapshotKind. I wonder if this is necessary to expose toplevel.

max-sixty commented 1 month ago

Thanks @lasernoises !

I tagged @mitsuhiko in case he has any views, will merge now and we can make changes later if needed.