Merge pull request #1185 from UziTech/eslint-all-the-things

Eslint all the things

.eslintignore

@@ -0,0 +1 @@
+*.min.js

bin/marked

@@ -5,9 +5,9 @@
  * Copyright (c) 2011-2013, Christopher Jeffrey (MIT License)
  */
 
-var fs = require('fs')
-  , util = require('util')
-  , marked = require('../');
+var fs = require('fs'),
+    path = require('path'),
+    marked = require('../');
 
 /**
  * Man Page
@@ -23,9 +23,9 @@ function help() {
     customFds: [0, 1, 2]
   };
 
-  spawn('man', [__dirname + '/../man/marked.1'], options)
-    .on('error', function(err) {
-      fs.readFile(__dirname + '/../man/marked.1.txt', 'utf8', function(err, data) {
+  spawn('man', [path.resolve(__dirname, '/../man/marked.1')], options)
+    .on('error', function() {
+      fs.readFile(path.resolve(__dirname, '/../man/marked.1.txt'), 'utf8', function(err, data) {
         if (err) throw err;
         console.log(data);
       });
@@ -37,13 +37,13 @@ function help() {
  */
 
 function main(argv, callback) {
-  var files = []
-    , options = {}
-    , input
-    , output
-    , arg
-    , tokens
-    , opt;
+  var files = [],
+      options = {},
+      input,
+      output,
+      arg,
+      tokens,
+      opt;
 
   function getarg() {
     var arg = argv.shift();
@@ -146,8 +146,8 @@ function main(argv, callback) {
  */
 
 function getStdin(callback) {
-  var stdin = process.stdin
-    , buff = '';
+  var stdin = process.stdin,
+      buff = '';
 
   stdin.setEncoding('utf8');
 

package.json

@@ -43,9 +43,9 @@
     "test:specs": "npm test -- test/specs/**/*-spec.js",
     "test:integration": "npm test -- test/integration/**/*-spec.js",
     "test:old": "node test",
-    "test:lint": "eslint lib/marked.js test/index.js",
+    "test:lint": "eslint bin/marked .",
     "bench": "node test --bench",
-    "lint": "eslint --fix lib/marked.js test/index.js",
+    "lint": "eslint --fix bin/marked .",
     "build": "uglifyjs lib/marked.js -cm  --comments /Copyright/ -o marked.min.js",
     "preversion": "npm run build && (git diff --quiet || git commit -am 'minify')"
   },

test/browser/test.js

@@ -1,11 +1,11 @@
-;(function() {
 
-var console = {}
-  , files = __TESTS__;
+;(function() {
+  var console = {},
+      files = __TESTS__; // eslint-disable-line no-undef
 
   console.log = function(text) {
-  var args = Array.prototype.slice.call(arguments, 1)
-    , i = 0;
+    var args = Array.prototype.slice.call(arguments, 1),
+        i = 0;
 
     text = text.replace(/%\w/g, function() {
       return args[i++] || '';
@@ -17,8 +17,8 @@ console.log = function(text) {
 
   if (!Object.keys) {
     Object.keys = function(obj) {
-    var out = []
-      , key;
+      var out = [],
+          key;
 
       for (key in obj) {
         if (Object.prototype.hasOwnProperty.call(obj, key)) {
@@ -31,19 +31,22 @@ if (!Object.keys) {
   }
 
   if (!Array.prototype.forEach) {
+    // eslint-disable-next-line no-extend-native
     Array.prototype.forEach = function(callback, context) {
       for (var i = 0; i < this.length; i++) {
-      callback.call(context || null, this[i], i, obj);
+        callback.call(context || null, this[i], i, this);
       }
     };
   }
 
   if (!String.prototype.trim) {
+    // eslint-disable-next-line no-extend-native
     String.prototype.trim = function() {
       return this.replace(/^\s+|\s+$/g, '');
     };
   }
 
+  // eslint-disable-next-line no-unused-vars
   function load() {
     return files;
   }
@@ -57,8 +60,7 @@ function escape(html, encode) {
       .replace(/'/g, '&#39;');
   }
 
-__LIBS__;
-
-(__MAIN__)();
+  __LIBS__; // eslint-disable-line no-undef, no-unused-expressions
 
+  (__MAIN__)(); // eslint-disable-line no-undef
 }).call(this);

test/integration/marked-spec.js

@@ -6,13 +6,12 @@ it('should run the test', function () {
 
 // http://spec.commonmark.org/0.28/#example-230
 it('should start an ordered list at 0 when requested', function () {
-  expect(
-    marked('0. ok')).
-    toBe("<ol start=\"0\">\n<li>ok</li>\n</ol>\n")
+  expect(marked('0. ok'))
+    .toBe('<ol start="0">\n<li>ok</li>\n</ol>\n')
 });
 
 // http://spec.commonmark.org/0.28/#example-234
 it('indents code within an explicitly-started ordered list', function () {
-  expect(marked("  10.  foo\n\n           bar")).
-  toBe("<ol start=\"10\">\n<li><p>foo</p>\n<pre><code>bar\n</code></pre></li>\n</ol>\n");
+  expect(marked('  10.  foo\n\n           bar'))
+    .toBe('<ol start="10">\n<li><p>foo</p>\n<pre><code>bar\n</code></pre></li>\n</ol>\n');
 });