Syntax for ApplicativeError

[davegurnell]

I've added syntax for the methods I mentioned in #561.

I'm happy with raiseError, handleError, handleErrorWith, recover, and recoverWith. However, I'm not sure about attemptT because the tests in ApplicativeErrorTests.scala required a bunch of heavyweight type hinting.

My inclination is to remove attempt and attemptT and leave the other methods intact. What do you think?

[ceedubs]

This doesn't appear to be used. Please tell me that it's mere presence doesn't affect whether the next line compiles? :|

[ceedubs]

@davegurnell thank you!

I'm probably missing something but the tests don't really appear to have more type gymnastics for attempt and attemptT than for other methods to me. Could you elaborate on the issue with those?

👍 either with or without attempt and attemptT

[davegurnell]

I guess it was late when I wrote the PR. The type annotations were on the expected value in the test for attemptT, not on the use of attemptT itself. I reckon this is good to go now.

[adelbertc]

In light of the fixed build (#954) this need to be re-merged against the latest master. Sorry about that!

[codecov-io]

Current coverage is 88.88%

Merging #947 into master will decrease coverage by -0.05% as of 291ebf3

@@            master    #947   diff @@
======================================
  Files          179     180     +1
  Stmts         2133    2142     +9
  Branches        42      42       
  Methods          0       0       
======================================
+ Hit           1897    1904     +7
  Partial          0       0       
- Missed         236     238     +2

Review entire Coverage Diff as of 291ebf3

Powered by Codecov. Updated on successful CI builds.

[adelbertc]

👍