rules: don't check cases for deps: V8 commits (#62)

Author: joyeecheung

lib/rules/title-format.js

@@ -52,17 +52,20 @@ module.exports = {
       }
     }
 
-    const result = /^([^:]+?): [A-Z]/.exec(context.title)
-    if (result) {
-      context.report({
-        id: id
-      , message: 'First word after subsystem(s) in title should be lowercase.'
-      , string: context.title
-      , line: 0
-      , column: result[1].length + 3
-      , level: 'fail'
-      })
-      pass = false
+    const isV8 = context.title.startsWith('deps: V8:')
+    if (!isV8) {
+      const result = /^([^:]+?): [A-Z]/.exec(context.title)
+      if (result) {
+        context.report({
+          id: id
+        , message: 'First word after subsystem(s) in title should be lowercase.'
+        , string: context.title
+        , line: 0
+        , column: result[1].length + 3
+        , level: 'fail'
+        })
+        pass = false
+      }
     }
 
     if (pass) {

test/rules/title-format.js

@@ -115,5 +115,23 @@ test('rule: title-format', (t) => {
     tt.end()
   })
 
+  t.test('Skip case checks for V8 updates ', (tt) => {
+    tt.plan(2)
+    const context = makeCommit('deps: V8: cherry-pick e0a109c')
+
+    context.report = (opts) => {
+      tt.pass('called report')
+      tt.strictSame(opts, {
+        id: 'title-format'
+      , message: 'Title is formatted correctly.'
+      , string: ''
+      , level: 'pass'
+      })
+    }
+
+    Rule.validate(context)
+    tt.end()
+  })
+
   t.end()
 })