ok_unwrap lint

[clarfonthey]

Right now the result_unwrap and option_unwrap lints are allow by default, meaning that result.ok().unwrap() will not be linted, even though result.ok().expect(...) is.

We could also do the same for result.err().unwrap(), suggesting unwrap_err.

This seems like a problem that should be addressed with a new lint which is allow by default.

[clarfonthey]

Note: I added a pull request to upstream to suggest adding an expect_err method to Result, allowing us to also lint result.err().expect(...).

[b-NC]

@rustbot claim