fix: linting on TravisCI and add new Node.js version to the testing matrix (#76)

realityking · gajus · commit 8d1a44e1eee7 · 2018-09-04T08:41:04.000+03:00
* Fix a linting error to make CI work again

* Expand the Node.js versions tests are run on

* fix: drop node v4, v6

BREAKING CHANGE:
diff --git a/.travis.yml b/.travis.yml
@@ -6,9 +6,8 @@ script:
   - npm run lint
   - npm run test
 node_js:
-  - 6
-  - 5
-  - 4
+  - 10
+  - 8
 notifications:
   email: false
 sudo: false
diff --git a/src/validateTableData.js b/src/validateTableData.js
@@ -37,6 +37,7 @@ export default (rows) => {
     // @todo Make an exception for newline characters.
     // @see https://github.com/gajus/table/issues/9
     for (const cell of cells) {
+      // eslint-disable-next-line no-control-regex
       if (/[\u0001-\u001A]/.test(cell)) {
         throw new Error('Table data must not contain control characters.');
       }

Original file line number	Diff line number	Diff line change
`@@ -37,6 +37,7 @@ export default (rows) => {`
`37`	`37`	`// @todo Make an exception for newline characters.`
`38`	`38`	`// @see https://github.com/gajus/table/issues/9`
`39`	`39`	`for (const cell of cells) {`
	`40`	`+ // eslint-disable-next-line no-control-regex`
`40`	`41`	`if (/[\u0001-\u001A]/.test(cell)) {`
`41`	`42`	`throw new Error('Table data must not contain control characters.');`
`42`	`43`	`}`