Merge pull request #1362 from LLazyEmail/1353-enable-rules-for-eslint

fix eslint end remove two rules

Author: vadim9999

.eslintignore

@@ -1,2 +1,3 @@
 src/parserMDReact/tests/_generated/
-src/constants/regexp/tests
+src/constants/regexp/tests
+.eslintrc.js

eslintrc.js renamed to .eslintrc.js

@@ -23,7 +23,7 @@ module.exports = {
     sourceType: 'module',
   },
   rules: {
-    'import/no-unresolved': 0,
+    // 'import/no-unresolved': 0,
     'import/no-dynamic-require': 0,
     'global-require': 0,
     'no-param-reassign': 0,
@@ -34,7 +34,6 @@ module.exports = {
     "quotes": 0,
     'comma-dangle': 0,
     'no-useless-escape': 0,
-    'no-unused-vars': 0,
     'no-tabs': 0,
     'no-underscore-dangle': 0, // maybe fix it later and remove _header variables
     'import/no-cycle': 0, // TODO https://github.com/LLazyEmail/markdown-to-email/issues/1286

package.json

@@ -14,8 +14,8 @@
     "emptyT": "npm run build && cross-env PARSE=plainEmpty node ./dist/bundle",
     "defaultT": "npm run build && cross-env PARSE=plainTemplate node ./dist/bundle",
     "test:react": "npm run checkFolders && jest ./src/parserMDReact/tests",
-    "lint:fix": "eslint -c ./eslintrc.js ./src --fix --ignore-pattern \"./package.json\" ",
-    "lint": "eslint -c  ./eslintrc.js ./src --quiet --ignore-pattern  \"./package.json\" ",
+    "lint:fix": "eslint -c ./.eslintrc.js ./src --fix --ignore-pattern \"./package.json\" ",
+    "lint": "eslint -c  ./.eslintrc.js ./src --quiet --ignore-pattern  \"./package.json\" ",
     "build:small": "npm run checkFolders && node ./src/index.js",
     "checkFolders": "node ./checkFolders",
     "test": "npm run checkFolders && jest ./src/tests",

src/callbacks/helpers.js

@@ -1,13 +1,10 @@
-import { mapKeys } from 'lodash';
 import { catchErrorTraceOutput } from '../domain/error-handle/index';
 
 // I think this method will be deleted or we'll update our
 // current version like W3_getWrapper
 function getWrapper(name, config) {
   const { layouts, folder } = config;
 
-  // console.log(layouts);
-
   try {
     const wrapper = layouts[folder][name];
     return wrapper;
@@ -17,10 +14,7 @@ function getWrapper(name, config) {
   return false;
 }
 
-// console.log(wrapper);
-
 // here we can have a problem, because we're passing more information into a config that it was before
-
 const _loopForWrapper = (config, component) => {
   try {
     return component(config);

src/callbacks/html/methods/br.js

@@ -1,6 +1,6 @@
 import { newLine } from '../helpers';
 
-import { commonReplace } from '../../../domain/replace-wrapper3.0';
+// import { commonReplace } from '../../../domain/replace-wrapper3.0';
 
 // TODO figure out `text` argument
 function _br(text, newLines) {

src/callbacks/html/methods/header.js

@@ -1,4 +1,7 @@
-import { newLine, beforeBeginingNewLine } from '../helpers';
+import {
+  newLine,
+  // beforeBeginingNewLine
+} from '../helpers';
 
 import { commonReplace } from '../../../domain/replace-wrapper3.0';
 

src/callbacks/html/methods/italic.js

@@ -1,6 +1,12 @@
 import { commonReplace } from '../../../domain/replace-wrapper3.0';
 
-const _italic = (text, content, a, b, c) => {
+const _italic = (
+  text,
+  // content,
+  // a,
+  // b,
+  // c
+) => {
   // TODO we need to debug italic
   // https://github.com/LLazyEmail/markdown-to-email/issues/1265
   // console.log(text)
@@ -22,7 +28,10 @@ const _italic = (text, content, a, b, c) => {
   return replaced;
 };
 
-const _italicAsterix = (text, content) => {
+const _italicAsterix = (
+  text,
+  // content
+) => {
   // console.log(text);
   // console.log(content);
 

src/callbacks/html/methods/lists.js

@@ -1,8 +1,8 @@
 import {
   // replaceWrapper,
   newLine,
-  beforeBeginingNewLine,
-  beforeEnd,
+  // beforeBeginingNewLine,
+  // beforeEnd,
 } from '../helpers';
 
 import { REGEXP_SUB_LISTS } from '../../../domain/regular-expressions';

src/callbacks/html/methods/separator.js

@@ -2,7 +2,10 @@ import { newLine } from '../helpers';
 
 import { commonReplace } from '../../../domain/replace-wrapper3.0';
 
-function _separator(text, content) {
+function _separator(
+  text,
+  // content
+) {
   // console.log(text);
   // console.log(content);
 

src/callbacks/html/methods/simple-without-replacer.js

@@ -1,5 +1,5 @@
 import { newLine } from '../helpers';
-import { commonReplace } from '../../../domain/replace-wrapper3.0';
+// import { commonReplace } from '../../../domain/replace-wrapper3.0';
 
 const _del = () => '<del>$1</del>';