You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
# This will deny use of unwrap on Result and Option outside of testsunwrap_used = { level = "deny", allow-unwrap-in-tests = "true" }
to the lints.clippy section of cargo.toml to enable denying unwrap in our code.
There are some test helpers which are not explicitly tests and will now pass. Easy enough to allow unwrap on their mod definitions though to unblock. Similar for our main fn in build.rs which is easy to resolve.
There are still ~60 uses of unwrap to resolve though so its not job done and we should reach some consensus before doing more than this exploration I think.
The text was updated successfully, but these errors were encountered:
Consider adding
to the lints.clippy section of cargo.toml to enable denying unwrap in our code.
There are some test helpers which are not explicitly tests and will now pass. Easy enough to allow unwrap on their mod definitions though to unblock. Similar for our main fn in build.rs which is easy to resolve.
There are still ~60 uses of unwrap to resolve though so its not job done and we should reach some consensus before doing more than this exploration I think.
The text was updated successfully, but these errors were encountered: