We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Result.unwrap
Adding the lint to our crate and server code should help finding and going about them
diff --git a/limitador-server/src/main.rs b/limitador-server/src/main.rs -#![deny(clippy::all, clippy::cargo)] +#![deny(clippy::all, clippy::cargo, clippy::unwrap_used)] diff --git a/limitador/src/lib.rs b/limitador/src/lib.rs -#![deny(clippy::all, clippy::cargo)] +#![deny(clippy::all, clippy::cargo, clippy::unwrap_used)]
Not that all necessary have to go, but probably cleaning some of these up and possibly swapping them for .expect() would be desirable.
.expect()
Crate has ~68 of them, while the server code has ~29 calls to unwrap() across all features
unwrap()
This probably relates to #131
The text was updated successfully, but these errors were encountered:
These is dependency to #131 for doing this properly...
Sorry, something went wrong.
No branches or pull requests
Adding the lint to our crate and server code should help finding and going about them
Not that all necessary have to go, but probably cleaning some of these up and possibly swapping them for
.expect()
would be desirable.Crate has ~68 of them, while the server code has ~29 calls to
unwrap()
across all featuresThis probably relates to #131
The text was updated successfully, but these errors were encountered: