Build: Require strict mode in Node.js scripts via ESLint

So far, only browser-based JS files were required to be in strict mode (in the
function form). This commit adds such a requirement to Node.js scripts where
the global form is preferred. All Node.js scripts in sloppy mode were
converted to strict mode.

Closes gh-4499

diff --git a/.eslintrc-node.json b/.eslintrc-node.json
index 589144272ce4e865005369ea487d3c16ef3c9465..544a92cdc8a9cd2c3a141e8c0167b48f8dad7d2a 100644 (file)
--- a/.eslintrc-node.json
+++ b/.eslintrc-node.json
@@ -10,5 +10,9 @@
        "env": {
                "es6": true,
                "node": true
+       },
+
+       "rules": {
+               "strict": ["error", "global"]
        }
 }

diff --git a/Gruntfile.js b/Gruntfile.js
index 012ce95dba38bfbbe81b0510b832eabd4329bd9a..e740690afd85492ee405c4b538424b30b1264cbd 100644 (file)
--- a/Gruntfile.js
+++ b/Gruntfile.js
@@ -1,6 +1,6 @@
-module.exports = function( grunt ) {
-       "use strict";
+"use strict";
 
+module.exports = function( grunt ) {
        function readOptionalJSON( filepath ) {
                var stripJSONComments = require( "strip-json-comments" ),
                        data = {};

diff --git a/build/release.js b/build/release.js
index f666bf1160ee0c87d2cf0bf8ecafb70e01269d87..6011066ff0f40c311a2277e30286461aebc2e4f0 100644 (file)
--- a/build/release.js
+++ b/build/release.js
@@ -1,3 +1,5 @@
+"use strict";
+
 var fs = require( "fs" );
 
 module.exports = function( Release ) {

diff --git a/build/release/cdn.js b/build/release/cdn.js
index c65c53e8caa5507965dd1941df344469a947cb5e..3d96d01791833f550cac2b5aae10ba731e2487d8 100644 (file)
--- a/build/release/cdn.js
+++ b/build/release/cdn.js
@@ -1,3 +1,5 @@
+"use strict";
+
 var
        fs = require( "fs" ),
        shell = require( "shelljs" ),

diff --git a/build/release/dist.js b/build/release/dist.js
index 0b1e04a7f6f77582d990f442790d22e88c15adf8..0ca396811a4a9b4d2a9f8081f469a4b4b470a5ef 100644 (file)
--- a/build/release/dist.js
+++ b/build/release/dist.js
@@ -1,3 +1,5 @@
+"use strict";
+
 module.exports = function( Release, files, complete ) {
 
        var

diff --git a/build/tasks/build.js b/build/tasks/build.js
index d536f538c2642593d0476d573b7db4edb472bb43..63dfc1ee867d76996f410043f3dcdf6d42e82750 100644 (file)
--- a/build/tasks/build.js
+++ b/build/tasks/build.js
@@ -4,10 +4,9 @@
  * and includes/excludes specified modules
  */
 
-module.exports = function( grunt ) {
-
-       "use strict";
+"use strict";
 
+module.exports = function( grunt ) {
        var fs = require( "fs" ),
                requirejs = require( "requirejs" ),
                Insight = require( "insight" ),

diff --git a/build/tasks/dist.js b/build/tasks/dist.js
index fa6920c88949afd3dbeb43d3cea5e1b99a506b97..f19929b896886838e1e6fecf5d0fbf6c7d08e1d9 100644 (file)
--- a/build/tasks/dist.js
+++ b/build/tasks/dist.js
@@ -1,7 +1,6 @@
-module.exports = function( grunt ) {
-
-       "use strict";
+"use strict";
 
+module.exports = function( grunt ) {
        var     fs = require( "fs" ),
                filename = grunt.option( "filename" ),
                distpaths = [

diff --git a/build/tasks/node_smoke_tests.js b/build/tasks/node_smoke_tests.js
index e3d69db883a472b2106bca6586fa9719ce662577..40c49a4d98a83a404da180d84f230ca7fcdc7890 100644 (file)
--- a/build/tasks/node_smoke_tests.js
+++ b/build/tasks/node_smoke_tests.js
@@ -1,7 +1,6 @@
-module.exports = ( grunt ) => {
-
-       "use strict";
+"use strict";
 
+module.exports = ( grunt ) => {
        const fs = require( "fs" );
        const spawnTest = require( "./lib/spawn_test.js" );
        const testsDir = "./test/node_smoke_tests/";

diff --git a/build/tasks/qunit_fixture.js b/build/tasks/qunit_fixture.js
index 3f332dad03b288912c4311011d3f15984dfa978f..138ca662ddb7929065c22f4e2c8abde11f53c118 100644 (file)
--- a/build/tasks/qunit_fixture.js
+++ b/build/tasks/qunit_fixture.js
@@ -1,3 +1,5 @@
+"use strict";
+
 var fs = require( "fs" );
 
 module.exports = function( grunt ) {

diff --git a/build/tasks/sourcemap.js b/build/tasks/sourcemap.js
index 3f21b2afd07b1aebe39ae00051151c19a05b2e5c..509374f2ee4e26a8697607b3d48b10c92dc464c3 100644 (file)
--- a/build/tasks/sourcemap.js
+++ b/build/tasks/sourcemap.js
@@ -1,3 +1,5 @@
+"use strict";
+
 var fs = require( "fs" );
 
 module.exports = function( grunt ) {

diff --git a/build/tasks/testswarm.js b/build/tasks/testswarm.js
index 88e883d0f956b0c80df4ec1ece3f57e15b16b42d..d2653e0e057d2fab5247145db029275fd87d0c5a 100644 (file)
--- a/build/tasks/testswarm.js
+++ b/build/tasks/testswarm.js
@@ -1,7 +1,6 @@
-module.exports = function( grunt ) {
-
-       "use strict";
+"use strict";
 
+module.exports = function( grunt ) {
        grunt.registerTask( "testswarm", function( commit, configFile, projectName, browserSets,
                        timeout, testMode ) {
                var jobName, config, tests,