fix: rename 'ascii' rule to 'notAscii' rule (the meaning does not cha…

…nge)
isuke · Mar 7, 2018 · ad79a60 · ad79a60
1 parent 525e83b
commit ad79a60
Show file tree

Hide file tree

Showing 4 changed files with 8 additions and 8 deletions.
diff --git a/.git_consistent b/.git_consistent
@@ -65,7 +65,7 @@ subject:
   rules:
     firstLatter: lower
     dotAtEnd: false
-    ascii: false
+    notAscii: false
 issueLink:
   type: branch
   required: false
@@ -82,4 +82,4 @@ body:
   rules:
     firstLatter: upper
     dotAtEnd: true
-    ascii: false
+    notAscii: false
diff --git a/README.adoc b/README.adoc
@@ -366,8 +366,8 @@ link:https://github.com/isuke/git-consistent/blob/master/sample_type_list.adoc[s
 | need dot (`.`) input value's last
 | boolean
 
-| `ascii`
-| allow ASCII?
+| `notAscii`
+| Use not ascii symbols
 | boolean
 
 | `numberOnly`

diff --git a/lib/gen-config.js b/lib/gen-config.js
@@ -122,7 +122,7 @@ const generateDefinitionsFile = (definitionsFileName, answers, dryRun = false) =
     rules: {
       firstLatter: answers.subjectFirstLatter,
       dotAtEnd:    answers.subjectDotAtEnd,
-      ascii:       false,
+      notAscii:    false,
     }
   }
 
@@ -134,7 +134,7 @@ const generateDefinitionsFile = (definitionsFileName, answers, dryRun = false) =
     rules: {
       firstLatter: 'upper',
       dotAtEnd:    true,
-      ascii:       false,
+      notAscii:    false,
     }
   }
 

diff --git a/lib/main.js b/lib/main.js
@@ -112,9 +112,9 @@ const checkValue = (term, value, definition) => {
           errorMessages.push(`${term} must be number only.`)
         }
         break
-      case 'ascii':
+      case 'notAscii':
         if (!ruleSetting && /[\u30a0-\u30ff\u3040-\u309f\u3005-\u3006\u30e0-\u9fcf]/.test(value)) {
-          errorMessages.push(`${term} should only alphabet.`)
+          errorMessages.push(`${term} should only ascii symbols.`)
         }
         break
       default: