Enable breakout in functions reduceRightToOption and reduceRightTo.

[takayahilton]

now

import cats.data.NonEmptyList
import cats.implicits._

val notAllEven = NonEmptyList.of(2, 4, 6, 9, 10, 12, 14)
notAllEven.reduceMapA { a => println(a); if (a % 2 == 0) Some(a) else None }

14
12
10
9
6
4
2
res: Option[Int] = None

After applying this PR

import cats.data.NonEmptyList
import cats.implicits._

val notAllEven = NonEmptyList.of(2, 4, 6, 9, 10, 12, 14)
notAllEven.reduceMapA { a => println(a); if (a % 2 == 0) Some(a) else None }

2
4
6
9
res: Option[Int] = None

[takayahilton]

ref: #3015

[codecov-commenter]

Codecov Report

Merging #3498 into master will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master    #3498   +/-   ##
=======================================
  Coverage   91.75%   91.75%           
=======================================
  Files         383      383           
  Lines        8406     8409    +3     
  Branches      232      217   -15     
=======================================
+ Hits         7713     7716    +3     
  Misses        693      693           

[barambani]

What do you think about adding a couple of tests to make sure that this is enforced in future changes ? On the line of your examples

test("test") {
    val notAllEven = NonEmptyList.of(2, 4, 6, 9, 10, 12, 14)
    val out = mutable.ListBuffer[Int]()

    notAllEven.reduceMapA { a => out += a; if (a % 2 == 0) Some(a) else None }

    out.toList should ===(List(2, 4, 6, 9))
  }

May be also a stack safety check ? Thanks.

[barambani]

Nice cleanup here and below, thanks.

[barambani]

thanks 👍