From c6157ff29357e89034dba0e442bced0a2afc3117 Mon Sep 17 00:00:00 2001 From: Dillon Kearns Date: Fri, 28 Jan 2022 08:51:50 -0800 Subject: [PATCH] Use Never in constructor as suggested in docs to avoid no unused constructor review error. --- review/src/ReviewConfig.elm | 1 - src/Form.elm | 4 ++-- 2 files changed, 2 insertions(+), 3 deletions(-) diff --git a/review/src/ReviewConfig.elm b/review/src/ReviewConfig.elm index 08c3aa59..97e0713c 100644 --- a/review/src/ReviewConfig.elm +++ b/review/src/ReviewConfig.elm @@ -118,7 +118,6 @@ noUnusedRules = |> Rule.ignoreErrorsForFiles [ "src/Head/Twitter.elm" -- keeping unused for future use for spec API , "src/RoutePattern.elm" - , "src/Form.elm" -- ignored because false positive for unused for Phantom Builder type, see https://github.com/jfmengels/elm-review-unused/issues/60 ] , NoUnused.CustomTypeConstructorArgs.rule |> ignoreInTest diff --git a/src/Form.elm b/src/Form.elm index ee9e282b..41eb8f9d 100644 --- a/src/Form.elm +++ b/src/Form.elm @@ -1337,11 +1337,11 @@ withStringProperty ( key, value ) (Field field) = type Yes - = Yes + = Yes Never type No - = No + = No Never {-| -}