Fix '--line-numbers none'

Author: bfontaine

CHANGELOG.md

@@ -1,5 +1,9 @@
 # Grape Changelog
 
+## 0.5.1
+### CLI
+* Fix `--line-numbers none`
+
 ## 0.5.0 (2020/10/27)
 
 ### Library

src/grape/cli.clj

@@ -105,10 +105,11 @@
          :no-trailing-newlines? (:no-trailing-newlines options)
          :line-numbers          (cond
                                   (contains? options :line-numbers)
-                                  (keyword (:line-numbers options))
+                                  (when (not= "none" (:line-numbers options))
+                                    (keyword (:line-numbers options)))
 
                                   (:no-line-numbers options)
-                                  :none
+                                  nil
 
                                   (:all-line-numbers options)
                                   :all

test/grape/cli_test.clj

@@ -45,12 +45,12 @@
       :first ["--line-numbers" "first"]
       :all ["--line-numbers" "all"]
       ;; Aliases
-      :none ["--line-numbers" "none"]
+      nil ["--line-numbers" "none"]
       :all ["--all-line-numbers"]
       :all ["-n"]
-      :none ["-N"]
+      nil ["-N"]
       ;; Mix of aliases
-      :none ["--all-line-numbers" "--no-line-numbers"]
+      nil ["--all-line-numbers" "--no-line-numbers"]
       ;; Mix of --line-numbers with aliases
       :first ["--line-numbers" "first" "--all-line-numbers"]
       :first ["--line-numbers" "first" "--no-line-numbers"]