Alright, that seems reasonable. Thanks for the explanation.
@silver wrote in Skynet/discord-bot#43 (comment):
@cordlesscoder wrote in #43 (comment):
What leads you to say I'm writing code I don't…
This is an env flag set on deployment. It will only fail if you provide an invalid value for configuration. So, the situation is: the discord bot has been started with improper configuration.…
The major bump happened 2 months ago and the TOML crate is ridiculously popular - I believe it's reasonable enough to assume that any regressions have been found and fixed. The image dependencies…
What leads you to say I'm writing code I don't understand? I made every change that has semantic significance with intent, and I can justify it just fine.
The original code instead ignores errors - that is not better.
The RwLock can't be required, as the only extra capability it adds is getting a mutable reference to the Database - something your code doesn't do once, and something sqlx's Pool intentionally avoids.
https://github.com/toml-rs/toml/blob/main/crates/toml/CHANGELOG.md#092---2025-07-11 That fixes an infinite loop on certain invalid TOML inputs
I didn't do this, my TOML formatter did(taplo). That's the dark side of enabling format on save :(
It isn't available in stable rustfmt, it's still a nightly feature
Please double check this replacement logic for FitTo