Add Future instances to implicit scope

[travisbrown]

See discussion here. I've not built or run tests locally yet but I think it should work.

[codecov-commenter]

Codecov Report

Merging #3572 into master will decrease coverage by 0.02%.
The diff coverage is 83.33%.

@@            Coverage Diff             @@
##           master    #3572      +/-   ##
==========================================
- Coverage   91.33%   91.30%   -0.03%     
==========================================
  Files         386      386              
  Lines        8612     8618       +6     
  Branches      265      262       -3     
==========================================
+ Hits         7866     7869       +3     
- Misses        746      749       +3     

[barambani]

This should also resolve this #3536. Adding it for reference.

[barambani]

We could remove FutureInstances from here

cats/js/src/test/scala/cats/tests/FutureTests.scala

Line 18 in a075234

class FutureTests extends CatsSuite with FutureInstances {

and the import from here

cats/jvm/src/test/scala/cats/tests/FutureSuite.scala

Line 3 in a075234

import cats.instances.future._

now. Or you would prefer keep testing the instances through them ? Happy to go on either way 👍 .

[barambani]

Silly me :). The test changes are already in.
Reminder to self: never look at prs from the phone.

[cb372]

Once this is merged, it would be handy to have another RC release so I can bump the dependency in scalafix/build.sbt on #3566

[travisbrown]

Yes, I'm traveling at the moment but could probably publish an RC tonight if this gets merged before then.

[cb372]

@travisbrown Don't worry about it, I didn't realise we were publishing snapshots to Sonatype. I was able to use a snapshot. Enjoy your holiday!