allan2 / dotenvy

A well-maintained fork of the dotenv crate
MIT License
664 stars 40 forks source link

Fix clippy lints #23

Closed rillian closed 1 year ago

rillian commented 1 year ago

The second one is technically an incompatible API change. However, the whole point of the __Nonexhaustive variant pattern is to get callers to include a wildcard branch when matching, so you could argue it doesn't require a semver bump: anyone referring to that variant explicitly was doing it wrong. But I don't know if that's the normal expectation; this pattern is new to me.