ESLint: enable strict rule

.eslintrc.json

@@ -1,7 +1,4 @@
 {
-    "env": {
-        "node": true
-    },
     "rules": {
         "no-bitwise": 0,
         "curly": 2,
@@ -25,7 +22,7 @@
         "no-control-regex": 2,
         "no-regex-spaces": 2,
         "no-undef": 2,
-        "strict": 0,
+        "strict": 2,
         "no-unused-vars": [0, {"vars": "all", "args": "none"}],
         "semi": 2,
 
@@ -59,6 +56,7 @@
         "$": false,
 
         "window": false,
+        "console": false,
         "setTimeout": false,
         "clearTimeout": false,
 

Gruntfile.js

@@ -21,11 +21,11 @@
  *
  */
 
+/*eslint-env node */
 /*jslint node: true */
-
-module.exports = function (grunt) {
 'use strict';
 
+module.exports = function (grunt) {
     // load dependencies
     require('load-grunt-tasks')(grunt, {
         pattern: [

src/LiveDevelopment/MultiBrowserImpl/launchers/node/LauncherDomain.js

@@ -21,9 +21,8 @@
  *
  */
 
+/*eslint-env node */
 /*jslint node: true */
-
-(function () {
 "use strict";
 
 var open = require("opn");
@@ -69,5 +68,3 @@
 }
 
 exports.init = init;
-
-}());

src/LiveDevelopment/MultiBrowserImpl/transports/node/NodeSocketTransportDomain.js

@@ -21,9 +21,8 @@
  *
  */
 
+/*eslint-env node */
 /*jslint node: true */
-
-(function () {
 "use strict";
 
 var WebSocketServer = require("ws").Server,
@@ -243,5 +242,3 @@
 }
 
 exports.init = init;
-
-}());

src/extensibility/node/ExtensionManagerDomain.js

@@ -21,6 +21,7 @@
  *
  */
 
+/*eslint-env node */
 /*jslint node: true */
 
 "use strict";

src/extensibility/node/package-validator.js

@@ -21,6 +21,7 @@
  *
  */
 
+/*eslint-env node */
 /*jslint node: true, regexp: true */
 
 "use strict";

src/extensibility/node/spec/Installation.spec.js

@@ -21,6 +21,7 @@
  *
  */
 
+/*eslint-env node */
 /*jslint node: true */
 /*global expect, describe, it, beforeEach, afterEach */
 

src/extensibility/node/spec/Validation.spec.js

@@ -21,6 +21,7 @@
  *
  */
 
+/*eslint-env node */
 /*jslint node: true */
 /*global expect, describe, it, afterEach */
 

src/extensions/default/StaticServer/node/StaticServerDomain.js

@@ -21,9 +21,8 @@
  *
  */
 
+/*eslint-env node */
 /*jslint node: true */
-
-(function () {
 "use strict";
 
 var http     = require("http"),
@@ -476,5 +475,3 @@
 }
 
 exports.init = init;
-
-}());

src/filesystem/impls/appshell/node/CSharpWatcher.js

@@ -26,6 +26,7 @@
  * https://github.com/Microsoft/vscode/blob/314e122b16c5c1ca0288c8006e9c9c3039a51cd7/src/vs/workbench/services/files/node/watcher/win32/csharpWatcherService.ts
  */
 
+/*eslint-env node */
 /*jslint node: true */
 
 "use strict";

src/filesystem/impls/appshell/node/ChokidarWatcher.js

@@ -21,6 +21,7 @@
  *
  */
 
+/*eslint-env node */
 /*jslint node: true */
 
 "use strict";

src/filesystem/impls/appshell/node/FileWatcherDomain.js

@@ -21,6 +21,7 @@
  *
  */
 
+/*eslint-env node */
 /*jslint node: true */
 
 "use strict";

src/filesystem/impls/appshell/node/FileWatcherManager.js

@@ -21,6 +21,7 @@
  *
  */
 
+/*eslint-env node */
 /*jslint node: true */
 
 "use strict";

src/search/node/FindInFilesDomain.js

@@ -21,10 +21,9 @@
  *
  */
 
+/*eslint-env node */
 /*jslint node: true */
 /*global setImmediate */
-
-(function () {
 "use strict";
 
 var fs = require("fs"),
@@ -684,5 +683,3 @@
 }
 
 exports.init = init;
-
-}());

tasks/build.js

@@ -21,11 +21,11 @@
  *
  */
 
+/*eslint-env node */
 /*jslint node: true, regexp: true */
-
-module.exports = function (grunt) {
 "use strict";
 
+module.exports = function (grunt) {
     var child_process   = require("child_process"),
         http            = require("http"),
         https           = require("https"),

tasks/lib/common.js

@@ -21,11 +21,11 @@
  *
  */
 
+/*eslint-env node */
 /*jslint node: true */
-
-module.exports = function (grunt) {
 "use strict";
 
+module.exports = function (grunt) {
     var common      = {},
         path        = require("path"),
         _platform;

tasks/npm-install.js

@@ -21,11 +21,11 @@
  *
  */
 
+/*eslint-env node */
 /*jslint node: true */
-
-module.exports = function (grunt) {
 "use strict";
 
+module.exports = function (grunt) {
     var common  = require("./lib/common")(grunt),
         build   = require("./build")(grunt);
 

tasks/test.js

@@ -21,11 +21,11 @@
  *
  */
 
+/*eslint-env node */
 /*jslint node: true */
-
-module.exports = function (grunt) {
 "use strict";
 
+module.exports = function (grunt) {
     var common          = require("./lib/common")(grunt),
         child_process   = require("child_process"),
         q               = require("q"),

tasks/update-release-number.js

@@ -21,11 +21,11 @@
  *
  */
 
+/*eslint-env node */
 /*jslint node: true */
-
-module.exports = function (grunt) {
 "use strict";
 
+module.exports = function (grunt) {
     var common = require("./lib/common")(grunt),
         semver = require("semver");
 

tasks/write-config.js

@@ -21,11 +21,11 @@
  *
  */
 
+/*eslint-env node */
 /*jslint node: true */
-
-module.exports = function (grunt) {
 "use strict";
 
+module.exports = function (grunt) {
     var common  = require("./lib/common")(grunt),
         build   = require("./build")(grunt);
 

test/node/TestingDomain.js

@@ -21,6 +21,7 @@
  *
  */
 
+/*eslint-env node */
 /*jslint node: true */
 
 "use strict";