Implemented a new rule: NoIgnoringErrors

[jfmengels]

Hey all 👋
I've written a rule to detect when the contents of a Result.Err is ignored

a =
  case result of
    Err _ -> ...
    Ok data -> ...

In cases where you only care about executing something like sending an HTTP request and not caring about the result, you can pre-emptively map that Task's Result to something that shows you don't care about the result or don't care about the error message, which I've explained in the rule's documentation.

I'm very curious to see if this is something that is reasonable to enforce, so please try it out and let me know what you think, but I at least think it's a nice tool to find where you could improve the user experience or even find bugs.

Docs: https://elm-doc-preview.netlify.app/NoIgnoringErrors?repo=jfmengels%2Felm-review-no-ignoring-errors&version=master
Try it out: elm-review --template jfmengels/elm-review-no-ignoring-errors/preview
Repo: https://github.com/jfmengels/elm-review-no-ignoring-errors