chore: Update lint config and fix errors

Author: fernandopasik

.eslintrc

@@ -1,9 +1,15 @@
 {
-  "extends": [ "fernandopasik", "fernandopasik/es6", "fernandopasik/react" ],
+  "extends": [ "airbnb" ],
   "env": {
     "node": true
   },
   "parserOptions": {
     "sourceType": "script"
+  },
+  "rules": {
+    "strict": [ "error", "global" ],
+    "consistent-return": "off",
+    "jsx-a11y/href-no-hash": "off",
+    "react/jsx-filename-extension": "off",
   }
 }

package.json

@@ -32,15 +32,19 @@
   },
   "dependencies": {
     "camelize": "^1.0.0",
+    "eslint-config-airbnb": "^15.1.0",
+    "eslint-plugin-import": "^2.7.0",
+    "eslint-plugin-jsx-a11y": "^6.0.2",
+    "eslint-plugin-react": "^7.3.0",
     "except": "^0.1.3",
     "front-matter": "^2.1.2",
     "node-prismjs": "^0.1.0",
     "remarkable": "^1.7.1"
   },
   "devDependencies": {
     "eslint": "^3.19.0",
-    "eslint-config-fernandopasik": "^0.5.1",
     "jest": "^20.0.0",
+    "react": "^15.6.1",
     "watch": "^1.0.2"
   },
   "engines": {

src/build.js

@@ -1,4 +1,5 @@
 'use strict';
+
 const camelize = require('camelize');
 const except = require('except');
 
@@ -16,14 +17,13 @@ const except = require('except');
  * @returns {String}              - React Component
  */
 module.exports = function build(markdown) {
-
   let doImports = 'import React from \'react\';\n';
-  const
-    imports = markdown.attributes.imports || {},
-    jsx = markdown.html.replace(/class=/g, 'className=');
+  const imports = markdown.attributes.imports || {};
+  const jsx = markdown.html.replace(/class=/g, 'className=');
 
   const frontMatterAttributes = except(markdown.attributes, 'imports');
 
+  // eslint-disable-next-line no-restricted-syntax
   for (const variable in imports) {
     // eslint-disable-next-line no-prototype-builtins
     if (imports.hasOwnProperty(variable)) {

src/index.js

@@ -1,21 +1,18 @@
 'use strict';
 
-const
-  build = require('./build.js'),
-  parser = require('./parser.js');
+const build = require('./build.js');
+const parser = require('./parser.js');
 
 /**
  * Main function
  * @param   {String}  content   Markdown file content
  */
-module.exports = function (content) {
-
+module.exports = function loader(content) {
   const callback = this.async();
 
   parser
     .parse(content)
     .then(build)
     .then(component => callback(null, component))
     .catch(callback);
-
 };

src/parser.js

@@ -1,11 +1,11 @@
 'use strict';
 
-const
-  frontMatter = require('front-matter'),
-  Prism = require('node-prismjs'),
-  Remarkable = require('remarkable'),
-  escapeHtml = require('remarkable/lib/common/utils').escapeHtml,
-  md = new Remarkable();
+const frontMatter = require('front-matter');
+const Prism = require('node-prismjs');
+const Remarkable = require('remarkable');
+const escapeHtml = require('remarkable/lib/common/utils').escapeHtml;
+
+const md = new Remarkable();
 
 /**
  * Wraps the code and jsx in an html component
@@ -42,9 +42,8 @@ function parseCodeBlock(code, lang, langPrefix, highlight) {
     codeBlock = highlight(code, lang);
   }
 
-  const
-    langClass = !lang ? '' : `${langPrefix}${escape(lang, true)}`,
-    jsx = code;
+  const langClass = !lang ? '' : `${langPrefix}${escape(lang, true)}`;
+  const jsx = code;
 
   codeBlock = codeBlock
     .replace(/{/g, '{"{"{')
@@ -84,29 +83,28 @@ function parseMarkdown(markdown) {
         const language = Prism.languages[lang] || Prism.languages.autoit;
         return Prism.highlight(code, language);
       },
-      xhtmlOut: true
+      xhtmlOut: true,
     };
 
     md.set(options);
 
-    md.renderer.rules.fence_custom.render = (tokens, idx, options) => {
+    md.renderer.rules.fence_custom.render = (tokens, idx, opts) => {
       // gets tags applied to fence blocks ```react html
       const codeTags = tokens[idx].params.split(/\s+/g);
       return parseCodeBlock(
         tokens[idx].content,
         codeTags[codeTags.length - 1],
-        options.langPrefix,
-        options.highlight
+        opts.langPrefix,
+        opts.highlight,
       );
     };
 
     try {
       html = md.render(markdown.body);
-      resolve({ html, attributes: markdown.attributes });
+      return resolve({ html, attributes: markdown.attributes });
     } catch (err) {
       return reject(err);
     }
-
   });
 }
 
@@ -136,5 +134,5 @@ module.exports = {
   parse,
   parseCodeBlock,
   parseFrontMatter,
-  parseMarkdown
+  parseMarkdown,
 };

test/build.spec.js

@@ -1,25 +1,23 @@
 'use strict';
 
-const
-  fs = require('fs'),
-  path = require('path'),
-  build = require('../src/build.js'),
-  parser = require('../src/parser.js');
+const fs = require('fs');
+const path = require('path');
+const build = require('../src/build.js');
+const parser = require('../src/parser.js');
 
 describe('Build Component', () => {
-
   let component = '';
   const mdFile = path.join(__dirname, './examples/hello-world.md');
 
-  beforeAll(done => {
+  beforeAll((done) => {
     fs.readFile(mdFile, 'utf8', (err, data) => {
       if (err) {
         return done(err);
       }
 
       parser
         .parse(data)
-        .then(html => {
+        .then((html) => {
           component = build(html);
           done();
         })
@@ -39,5 +37,4 @@ describe('Build Component', () => {
   it('exports the front-matter attributes', () => {
     expect(component).toMatch(/export const attributes = {"testFrontMatter":"hello world"}/);
   });
-
 });

test/index.spec.js

@@ -1,16 +1,14 @@
 'use strict';
 
-const
-  fs = require('fs'),
-  path = require('path'),
-  loader = require('../src/index.js');
+const fs = require('fs');
+const path = require('path');
+const loader = require('../src/index.js');
 
 describe('React Markdown Loader', () => {
-
   let mdExample = '';
   const mdFile = path.join(__dirname, './examples/hello-world.md');
 
-  beforeAll(done => {
+  beforeAll((done) => {
     fs.readFile(mdFile, 'utf8', (err, data) => {
       if (err) {
         return done(err);
@@ -21,14 +19,13 @@ describe('React Markdown Loader', () => {
     });
   });
 
-  it('converts markdown to component', done => {
+  it('converts markdown to component', (done) => {
     // eslint-disable-next-line prefer-reflect
     loader.call({
-      async: err => {
+      async: (err) => {
         expect(err).not.toBeDefined();
         done();
-      }
+      },
     }, mdExample);
   });
-
 });

test/parser.spec.js

@@ -1,16 +1,14 @@
 'use strict';
 
-const
-  fs = require('fs'),
-  path = require('path'),
-  parser = require('../src/parser.js');
+const fs = require('fs');
+const path = require('path');
+const parser = require('../src/parser.js');
 
 describe('Parse Markdown', () => {
-
   let mdExample = '';
   const mdFile = path.join(__dirname, './examples/hello-world.md');
 
-  beforeAll(done => {
+  beforeAll((done) => {
     fs.readFile(mdFile, 'utf8', (err, data) => {
       if (err) {
         return done(err);
@@ -35,9 +33,8 @@ describe('Parse Markdown', () => {
   });
 
   it('example code blocks have run and source code', () => {
-    const
-      exampleCode = 'example',
-      result = parser.codeBlockTemplate(exampleCode, exampleCode);
+    const exampleCode = 'example';
+    const result = parser.codeBlockTemplate(exampleCode, exampleCode);
 
     expect(result).toEqual(`
 <div class="example">
@@ -51,21 +48,20 @@ describe('Parse Markdown', () => {
   });
 
   it('parses markdown with live code blocks', () =>
-    parser.parse(mdExample).then(result => {
+    parser.parse(mdExample).then((result) => {
       expect(result.html).toMatch(/<div class="run"><HelloWorld \/>\s*<Button label="Hello World" \/>\s*<\/div>/);
-    })
+    }),
   );
 
   it('parses markdown and created valid html for JSX', () =>
-    parser.parse('![](myImage.png)').then(result => {
+    parser.parse('![](myImage.png)').then((result) => {
       expect(result.html).toMatch(/<p><img src="myImage.png" alt="" \/><\/p>\n/);
-    })
+    }),
   );
 
   it('provides the front-matter attributes', () =>
-    parser.parse(mdExample).then(result => {
+    parser.parse(mdExample).then((result) => {
       expect(result.attributes).toHaveProperty('test-front-matter', 'hello world');
-    })
+    }),
   );
-
 });