Merge pull request #69 from miniBill/main

Run elm-review, simplify code

review/src/ReviewConfig.elm

@@ -39,30 +39,29 @@ config : List Rule
 config =
     [ NoMissingTypeExpose.rule
     , NoUnused.Dependencies.rule
-
-    -- , Docs.NoMissing.rule
-    --    { document = onlyExposed
-    --    , from = exposedModules
-    --    }
-    --, Docs.ReviewLinksAndSections.rule
-    --, Docs.ReviewAtDocs.rule
-    --, Docs.UpToDateReadmeLinks.rule
-    --, NoDebug.Log.rule
-    --, NoDebug.TodoOrToString.rule
-    --    |> Rule.ignoreErrorsForDirectories [ "tests/" ]
-    --, NoExposingEverything.rule
-    --, NoImportingEverything.rule []
-    --, NoMissingTypeAnnotation.rule
-    --, NoMissingTypeAnnotationInLetIn.rule
-    --, NoSimpleLetBody.rule
-    --, NoPrematureLetComputation.rule
-    --, NoUnused.CustomTypeConstructors.rule []
-    --, NoUnused.CustomTypeConstructorArgs.rule
-    --, NoUnused.Exports.rule
-    --, NoUnused.Parameters.rule
-    --, NoUnused.Patterns.rule
-    --, NoUnused.Variables.rule
-    --, Simplify.rule Simplify.defaults
+    , Docs.NoMissing.rule
+        { document = onlyExposed
+        , from = exposedModules
+        }
+    , Docs.ReviewLinksAndSections.rule
+    , Docs.ReviewAtDocs.rule
+    , Docs.UpToDateReadmeLinks.rule
+    , NoDebug.Log.rule
+    , NoDebug.TodoOrToString.rule
+        |> Rule.ignoreErrorsForDirectories [ "tests/" ]
+    , NoExposingEverything.rule
+    , NoImportingEverything.rule []
+    , NoMissingTypeAnnotation.rule
+    , NoMissingTypeAnnotationInLetIn.rule
+    , NoSimpleLetBody.rule
+    , NoPrematureLetComputation.rule
+    , NoUnused.CustomTypeConstructors.rule []
+    , NoUnused.CustomTypeConstructorArgs.rule
+    , NoUnused.Exports.rule
+    , NoUnused.Parameters.rule
+    , NoUnused.Patterns.rule
+    , NoUnused.Variables.rule
+    , Simplify.rule Simplify.defaults
     ]
         |> List.map
             (Rule.ignoreErrorsForDirectories

review/suppressed/Docs.NoMissing.json

@@ -0,0 +1,14 @@
+{
+  "version": 1,
+  "automatically created by": "elm-review suppress",
+  "learn more": "elm-review suppress --help",
+  "suppressions": [
+    { "count": 32, "filePath": "src/Elm/Case/Branch.elm" },
+    { "count": 29, "filePath": "src/Elm.elm" },
+    { "count": 22, "filePath": "src/Elm/Annotation.elm" },
+    { "count": 13, "filePath": "src/Elm/Case.elm" },
+    { "count": 9, "filePath": "src/Elm/Let.elm" },
+    { "count": 8, "filePath": "src/Elm/Declare.elm" },
+    { "count": 6, "filePath": "src/Elm/ToString.elm" }
+  ]
+}

review/suppressed/Docs.ReviewAtDocs.json

@@ -0,0 +1,8 @@
+{
+  "version": 1,
+  "automatically created by": "elm-review suppress",
+  "learn more": "elm-review suppress --help",
+  "suppressions": [
+    { "count": 2, "filePath": "src/Elm.elm" }
+  ]
+}

review/suppressed/Docs.ReviewLinksAndSections.json

@@ -0,0 +1,8 @@
+{
+  "version": 1,
+  "automatically created by": "elm-review suppress",
+  "learn more": "elm-review suppress --help",
+  "suppressions": [
+    { "count": 1, "filePath": "src/Elm.elm" }
+  ]
+}

review/suppressed/Docs.UpToDateReadmeLinks.json

@@ -0,0 +1,8 @@
+{
+  "version": 1,
+  "automatically created by": "elm-review suppress",
+  "learn more": "elm-review suppress --help",
+  "suppressions": [
+    { "count": 2, "filePath": "README.md" }
+  ]
+}

review/suppressed/NoUnused.CustomTypeConstructorArgs.json

@@ -0,0 +1,9 @@
+{
+  "version": 1,
+  "automatically created by": "elm-review suppress",
+  "learn more": "elm-review suppress --help",
+  "suppressions": [
+    { "count": 1, "filePath": "src/Elm/Op.elm" },
+    { "count": 1, "filePath": "src/Internal/Compiler.elm" }
+  ]
+}

review/suppressed/NoUnused.CustomTypeConstructors.json

@@ -0,0 +1,9 @@
+{
+  "version": 1,
+  "automatically created by": "elm-review suppress",
+  "learn more": "elm-review suppress --help",
+  "suppressions": [
+    { "count": 4, "filePath": "src/Internal/Compiler.elm" },
+    { "count": 1, "filePath": "src/Internal/Comments.elm" }
+  ]
+}

review/suppressed/NoUnused.Exports.json

@@ -0,0 +1,9 @@
+{
+  "version": 1,
+  "automatically created by": "elm-review suppress",
+  "learn more": "elm-review suppress --help",
+  "suppressions": [
+    { "count": 1, "filePath": "src/Error/Format.elm" },
+    { "count": 1, "filePath": "src/Internal/Debug.elm" }
+  ]
+}

review/suppressed/NoUnused.Parameters.json

@@ -0,0 +1,8 @@
+{
+  "version": 1,
+  "automatically created by": "elm-review suppress",
+  "learn more": "elm-review suppress --help",
+  "suppressions": [
+    { "count": 1, "filePath": "src/Internal/Compiler.elm" }
+  ]
+}

review/suppressed/NoUnused.Variables.json

@@ -0,0 +1,17 @@
+{
+  "version": 1,
+  "automatically created by": "elm-review suppress",
+  "learn more": "elm-review suppress --help",
+  "suppressions": [
+    { "count": 9, "filePath": "src/Internal/Compiler.elm" },
+    { "count": 5, "filePath": "src/Elm.elm" },
+    { "count": 5, "filePath": "src/Internal/Write.elm" },
+    { "count": 4, "filePath": "src/Internal/Debug.elm" },
+    { "count": 2, "filePath": "src/Elm/Op.elm" },
+    { "count": 2, "filePath": "src/Internal/Comments.elm" },
+    { "count": 2, "filePath": "tests/Pattern.elm" },
+    { "count": 1, "filePath": "src/Internal/Index.elm" },
+    { "count": 1, "filePath": "tests/Elm/Expect.elm" },
+    { "count": 1, "filePath": "tests/PackageHelpers.elm" }
+  ]
+}

review/suppressed/Simplify.json

@@ -0,0 +1,8 @@
+{
+  "version": 1,
+  "automatically created by": "elm-review suppress",
+  "learn more": "elm-review suppress --help",
+  "suppressions": [
+    { "count": 2, "filePath": "src/Internal/Compiler.elm" }
+  ]
+}