JSHint → ESLint (#279)

Author: AlekseyMartynov

.eslintignore

@@ -0,0 +1,5 @@
+docs
+js-test/check-ts-compilation.*
+js-test/test-bundle.js
+net
+node_modules

.eslintrc.json

@@ -0,0 +1,14 @@
+{
+  "globals": {
+    "define": false,
+    "module": false,
+    "require": false
+  },
+  "rules": {
+    "eqeqeq": [ "error", "allow-null" ],
+    "no-undef": "error",
+    "no-unused-expressions": "error",
+    "no-unused-vars": "error",
+    "strict": [ "error", "safe" ]
+  }
+}

appveyor.yml

@@ -21,7 +21,7 @@ build_script:
 
 test_script:
   - npm run eclint
-  - npm run jshint
+  - npm run eslint
 
   - npm i --no-save "devextreme@~16.2.0"
   - npm run karma

build/.eslintrc.json

@@ -0,0 +1,8 @@
+{
+  "env": {
+    "node": true
+  },
+  "parserOptions": {
+    "ecmaVersion": 6
+  }
+}

build/make-nojquery.js

@@ -1,4 +1,4 @@
-// jshint esversion:6, node:true
+"use strict";
 
 const fs = require("fs");
 const path = require("path");

build/replace-meta.js

@@ -1,5 +1,3 @@
-// jshint node:true, esversion:6
-
 "use strict";
 
 const fs = require("fs");
@@ -20,7 +18,7 @@ const REPO_NAME = process.argv[4];
 let META_VERSION_NUMERIC = "99.0.0";
 let META_VERSION_FULL;
 
-if(REPO_NAME == MAIN_REPO_NAME && /^v?(([.\d]+)[\w-]*)$/.test(TAG)) {
+if(REPO_NAME === MAIN_REPO_NAME && /^v?(([.\d]+)[\w-]*)$/.test(TAG)) {
     META_VERSION_FULL = RegExp.$1;
     META_VERSION_NUMERIC = RegExp.$2;
 } else if(BUILD_NUMBER) {

js-test/ie-stubs.js

@@ -1,3 +1,6 @@
+/* eslint-env browser */
+/* eslint strict: off */
+
 if(!Object.setPrototypeOf)
     Object.setPrototypeOf = function() { };
 

js-test/karma.conf.js

@@ -1,3 +1,7 @@
+/* eslint-env node */
+
+"use strict";
+
 module.exports = function(config) {
     var options = {
         basePath: "",

js-test/test.js

@@ -1,5 +1,5 @@
-// jshint strict: true, browser: true, undef: true, unused: true, eqeqeq: true
-/* global ASPNET_DATA_SCRIPT, DevExpress, define, module, Promise, require */
+/* eslint-env browser */
+/* global ASPNET_DATA_SCRIPT:false, DevExpress:false, Promise:false */
 
 (function(factory) {
     "use strict";

js/dx.aspnet.data.js

@@ -1,8 +1,7 @@
 // https://github.com/DevExpress/DevExtreme.AspNet.Data
 // Copyright (c) Developer Express Inc.
 
-// jshint strict: true, undef: true, unused: true, eqeqeq: true
-/* global DevExpress, jQuery, define, module, require */
+/* global DevExpress:false, jQuery:false */
 
 (function(factory) {
     "use strict";