rust-analyzer / smol_str

Apache License 2.0
409 stars 53 forks source link

Changelog / release notes #56

Open alexheretic opened 1 year ago

alexheretic commented 1 year ago

A breaking bump to 0.2.0 was published recently. However, I can't see any notes on why in

Looking at the commits it seems feature optional changes & From impls are technically breaking though should be fine for most usages. But having to read commit messages for this isn't ideal as a user of this lib.

Is there somewhere else I should be looking?

Veykril commented 1 year ago

Ah sorry, what roughly changed is is that the serde dependency is no longer pulled in by default (it is now a proper optional dependency) and the From impls have changed, https://github.com/rust-analyzer/smol_str/commit/2a2b7df56861d98307b83ef8f8861e01bdfb9142, they are less general now (no longer anything that implements AsRef<str> is now From/Intoable), this had to be done so we could offer an AsRef<str> impl.