tc39 · oyilmaztekin · Jul 1, 2019 · Jul 2, 2019 · Jul 2, 2019 · Jul 2, 2019
diff --git a/tools/markdown-checker/.eslintignore b/tools/markdown-checker/.eslintignore
@@ -0,0 1 @@
 *_tests_*
diff --git a/tools/markdown-checker/.eslintrc.json b/tools/markdown-checker/.eslintrc.json
@@ -0,0 1,21 @@
 {
  "env": {
  "node": true,
  "es6": true
  },
  "extends": [
  "airbnb-base"
  ],
  "globals": {
  "Atomics": "readonly",
  "SharedArrayBuffer": "readonly"
  },
  "parserOptions": {
  "ecmaVersion": 2018
  },
  "rules": {
  "no-underscore-dangle": 0,
  "import/no-extraneous-dependencies": 0,
  "no-unused-vars": 0
  }
 }
diff --git a/tools/markdown-checker/index.js b/tools/markdown-checker/index.js
@@ -1,8 1,8 @@
-const readMarkdown = require("./lib/parser/readMarkdown");
-const parseToAST = require("./lib/parser/parseToAst");
-const collectLinkDefinitions = require("./lib/analyzer/collectLinkDefinitions");
-const tableAnalyzer = require("./lib/analyzer/analyzeTable");
-const config = require("./config.json");
 const readMarkdown = require('./lib/parser/readMarkdown');
 const parseToAST = require('./lib/parser/parseToAst');
 const collectLinkDefinitions = require('./lib/analyzer/collectLinkDefinitions');
 const tableAnalyzer = require('./lib/analyzer/analyzeTable');
 const config = require('./config.json');
 
 function processStage3({ stage3 } = config) {
  const markdownStage3 = readMarkdown(stage3);

diff --git a/tools/markdown-checker/lib/analyzer/analyzeTable.js b/tools/markdown-checker/lib/analyzer/analyzeTable.js
@@ -4,8 4,8 @@
  *
  */
 
-const detectTables = require("./detectTables");
-const detectHeader = require("./detectHeaders");
 const detectTables = require('./detectTables');
 const detectHeader = require('./detectHeaders');
 
 /**
  *

diff --git a/tools/markdown-checker/lib/analyzer/collectLinkDefinitions.js b/tools/markdown-checker/lib/analyzer/collectLinkDefinitions.js
@@ -2,14 2,15 @@
  * @param {Object} AST
  * @returns {Object} - collected link definitions
  */
-module.exports = function(AST) {
- let definitions = {};
- AST.children.forEach(node => {
- if (node.type === "definition") {
- let label = node.label;
- definitions[label] = node.url;
- }
- });
- return definitions;
- };
-
 function collectLinkDefinitions(AST) {
  const definitions = {};
  AST.children.forEach((node) => {
  if (node.type === 'definition') {
  const { label } = node;
  definitions[label] = node.url;
  }
  });
  return definitions;
 }
 
 module.exports = collectLinkDefinitions;
diff --git a/tools/markdown-checker/lib/analyzer/detectHeaders.js b/tools/markdown-checker/lib/analyzer/detectHeaders.js
@@ -1,10 1,10 @@
-const traverseChildren = require("./../transformer/traverser").traverseChildren;
 const { traverseChildren } = require('./../transformer/traverser');
 /**
  * @param {Object} tableRow
- * @returns {Object} - collected header template 
  * @returns {Object} - collected header template
  */
 module.exports = function detectHeaders(tableRow) {
  let cellSize = tableRow.align.length;
- let [headerNode] =  tableRow.children;
- //create header template here
  let [headerNode] = tableRow.children;
  // create header template here
 };
diff --git a/tools/markdown-checker/lib/mocks/mock.js b/tools/markdown-checker/lib/mocks/mock.js
diff --git a/tools/markdown-checker/lib/parser/__tests__/readMarkdown.test.js b/tools/markdown-checker/lib/parser/__tests__/readMarkdown.test.js
@@ -1,14 1,14 @@
-const readMarkdown = require("./../readMarkdown");
-const mock = require("../../mocks/mock");
-const config = require("./../../../config.json");
 const readMarkdown = require('./../readMarkdown');
 const mock = require('../../mocks/mock');
 const config = require('./../../../config.json');
 
 const sourceCode = readMarkdown(config.stage3);
-describe("testing parser", () => {
- test("is defined", () => {
 describe('testing parser', () => {
  test('is defined', () => {
  expect(sourceCode).toBeDefined();
  });
 
- test("result is matched to given mock AST", () => {
  test('result is matched to given mock AST', () => {
  expect(sourceCode).toEqual(mock);
  });
 });
diff --git a/tools/markdown-checker/lib/parser/parseToAst.js b/tools/markdown-checker/lib/parser/parseToAst.js
@@ -1,12 1,14 @@
-const unified = require("unified");
-const markdown = require("remark-parse");
 const unified = require('unified');
 const markdown = require('remark-parse');
 
 /**
  * @param {string} file - file path
- * @returns {Object} - returns parsed markdown file as an AST 
  * @returns {Object} - returns parsed markdown file as an AST
  */
-module.exports = function(file) {
 function parseToAst(file) {
  return unified()
  .use(markdown)
  .parse(file);
-};
 }
 
 module.exports = parseToAst;
diff --git a/tools/markdown-checker/lib/parser/readMarkdown.js b/tools/markdown-checker/lib/parser/readMarkdown.js
@@ -1,11 1,13 @@
-const fs = require("fs");
-const path = require("path");
 const fs = require('fs');
 const path = require('path');
 
 /**
  * @param {string} markdownPath - path of the markdown file
  * @returns {string} - source code of the markdown as a string
  */
-module.exports = function(markdownPath) {
 function readMarkdown(markdownPath) {
  const resolvedPath = path.resolve(__dirname, markdownPath);
- return fs.readFileSync(resolvedPath, "utf-8");
-};
  return fs.readFileSync(resolvedPath, 'utf-8');
 }
 
 module.exports = readMarkdown;
diff --git a/tools/markdown-checker/lib/transformer/traverser.js b/tools/markdown-checker/lib/transformer/traverser.js
@@ -1,10 1,10 @@
 /**
- * @param {Object} Node 
- * @param {function} callBackLogic 
  * @param {Object} Node
  * @param {function} callBackLogic
  */
 
- module.exports = function traverseChildren(node, callBackLogic) {
- node.forEach(n => {
 module.exports = function traverseChildren(node, callBackLogic) {
  node.forEach((n) => {
  callBackLogic();
  if (n.children) {
  traverseChildren(n);