Revert "rules: skip line-length rule for URLs and quoted lines (nodejs#30)"

This reverts commit 5ef341d.

Author: richardlau

lib/rules/line-length.js

@@ -31,12 +31,6 @@ module.exports = {
     var failed = false
     for (let i = 0; i < parsed.body.length; i++) {
       const line = parsed.body[i]
-      // Skip quoted lines, e.g. for original commit messages of V8 backports.
-      if (line.startsWith('    '))
-        continue
-      // Skip lines with URLs.
-      if (/https?:\/\//.test(line))
-        continue
       if (line.length > len) {
         failed = true
         context.report({

test/rules/line-length.js

@@ -67,69 +67,5 @@ ${'aaa'.repeat(30)}`
     tt.end()
   })
 
-  t.test('quoted lines', (tt) => {
-    const v = new Validator()
-    const context = new Commit({
-      sha: 'e7c077c610afa371430180fbd447bfef60ebc5ea'
-    , author: {
-        name: 'Evan Lucas'
-      , email: '[email protected]'
-      , date: '2016-04-12T19:42:23Z'
-      }
-    , message: `src: make foo mor foo-ey
-
-Here’s the original code:
-
-    ${'aaa'.repeat(30)}
-
-That was the original code.
-`
-    }, v)
-
-    context.report = (opts) => {
-      tt.pass('called report')
-      tt.equal(opts.id, 'line-length', 'id')
-      tt.equal(opts.string, '', 'string')
-      tt.equal(opts.level, 'pass', 'level')
-    }
-
-    Rule.validate(context, {
-      options: {
-        length: 72
-      }
-    })
-    tt.end()
-  })
-
-  t.test('URLs', (tt) => {
-    const v = new Validator()
-    const context = new Commit({
-      sha: 'e7c077c610afa371430180fbd447bfef60ebc5ea'
-    , author: {
-        name: 'Evan Lucas'
-      , email: '[email protected]'
-      , date: '2016-04-12T19:42:23Z'
-      }
-    , message: `src: make foo mor foo-ey
-
-https://${'very-'.repeat(80)}-long-url.org/
-`
-    }, v)
-
-    context.report = (opts) => {
-      tt.pass('called report')
-      tt.equal(opts.id, 'line-length', 'id')
-      tt.equal(opts.string, '', 'string')
-      tt.equal(opts.level, 'pass', 'level')
-    }
-
-    Rule.validate(context, {
-      options: {
-        length: 72
-      }
-    })
-    tt.end()
-  })
-
   t.end()
 })

test/validator.js

@@ -205,11 +205,11 @@ test('Validator - real commits', (t) => {
       const filtered = msgs.filter((item) => {
         return item.level === 'fail'
       })
-      tt.equal(filtered.length, 2, 'messages.length')
+      tt.equal(filtered.length, 3, 'messages.length')
       const ids = filtered.map((item) => {
         return item.id
       })
-      const exp = ['line-length', 'title-length']
+      const exp = ['line-length', 'line-length', 'title-length']
       tt.deepEqual(ids.sort(), exp.sort(), 'message ids')
       tt.end()
     })