rtfeldman#446 removed unused function isTodo

frankschmitt · Oct 27, 2020 · b25016a · b25016a
1 parent bc970d2
commit b25016a
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 13 deletions.
diff --git a/elm/review/src/ReviewConfig.elm b/elm/review/src/ReviewConfig.elm
@@ -35,8 +35,7 @@ config =
             [ "src/Test/Runner/Node/Vendor/Diff.elm"
             , "src/Test/Runner/Node.elm"
             , "src/Test/Runner/Node/Vendor/Console.elm" 
-            , "src/Test/Reporter/TestResults.elm"
-            --, "src/Console/Text.elm"
+            --, "src/Test/Reporter/TestResults.elm" -- isTodo
             ]
     , NoUnused.Modules.rule
     , NoUnused.CustomTypeConstructorArgs.rule

diff --git a/elm/src/Test/Reporter/TestResults.elm b/elm/src/Test/Reporter/TestResults.elm
@@ -4,7 +4,6 @@ module Test.Reporter.TestResults exposing
     , SummaryInfo
     , TestResult
     , isFailure
-    , isTodo
     , outcomesFromExpectations
     )
 
@@ -42,16 +41,6 @@ type alias Failure =
     }
 
 
-isTodo : Outcome -> Bool
-isTodo outcome =
-    case outcome of
-        Todo _ ->
-            True
-
-        _ ->
-            False
-
-
 isFailure : Outcome -> Bool
 isFailure outcome =
     case outcome of