remove the OptionT method, seems limited in value

typelevel · May 7, 2018 · 4da23e1 · 4da23e1
1 parent 6e05158
commit 4da23e1
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 11 deletions.
diff --git a/core/src/main/scala/cats/data/OptionT.scala b/core/src/main/scala/cats/data/OptionT.scala
@@ -137,14 +137,6 @@ final case class OptionT[F[_], A](value: F[Option[A]]) {
    * }}}
    */
   def toNested: Nested[F, Option, A] = Nested(value)
-
-  /**
-   * This repeats an F until we get defined values. This can be useful
-   * for polling type operations on State (or RNG) Monads, or in effect
-   * monads.
-   */
-  def untilDefined(implicit F: FlatMap[F]): F[A] =
-    (new syntax.FlatMapOptionOps(value)).untilDefinedM
 }
 
 object OptionT extends OptionTInstances {

diff --git a/tests/src/test/scala/cats/tests/IndexedStateTSuite.scala b/tests/src/test/scala/cats/tests/IndexedStateTSuite.scala
@@ -3,7 +3,7 @@ package tests
 
 import catalysts.Platform
 import cats.arrow.{Profunctor, Strong}
-import cats.data.{EitherT, IndexedStateT, OptionT, State, StateT}
+import cats.data.{EitherT, IndexedStateT, State, StateT}
 import cats.arrow.Profunctor
 import cats.kernel.instances.tuple._
 import cats.laws.discipline._
@@ -280,8 +280,6 @@ class IndexedStateTSuite extends CatsSuite {
     }
 
     counter.untilDefinedM.run(0).value should === ((stackSafeTestSize + 2, stackSafeTestSize + 1))
-
-    OptionT(counter).untilDefined.run(0).value should === ((stackSafeTestSize + 2, stackSafeTestSize + 1))
   }
 
   test("foreverM works") {