ESlint strict

.eslintrc

@@ -1,5 +1,8 @@
 {
-    "extends": "airbnb",
+	"extends": "airbnb-base",
+	"parserOptions": {
+		"sourceType": "script"
+	},
 
 	"rules": {
 		// Customized
@@ -16,10 +19,12 @@
 		"newline-per-chained-call": "off",
 		"no-console": "off",
 		"no-mixed-operators": ["error", { "allowSamePrecedence": true }],
+		"strict": ["error", "global"],
 
 		// ES6
 		"prefer-rest-params": "off",
 		"prefer-spread": "off",
+		"prefer-arrow-callback": "off",
 
 		// TODO
 		"import/no-unresolved": "off",
@@ -81,7 +86,6 @@
 		"no-tabs": "off",
 		"indent": "off",
 		"func-names": "off",
-        "prefer-arrow-callback": "off",
 		"object-curly-spacing": "off",
 		"no-var": "off",
 		"no-shadow": "off",
@@ -95,7 +99,8 @@
 		"object-shorthand": "off",
 		"no-param-reassign": "off",
 		"consistent-return": "off",
-        "strict": "off",
+		"no-multi-assign": "off",
+		// "strict": "off",
 		// "comma-dangle": "off",
 		// "no-multi-spaces": "off",
 		// "quotes": "off",

package.json

@@ -99,10 +99,8 @@
   "devDependencies": {
     "coveralls": "^2.11.14",
     "eslint": "^3.12.0",
-    "eslint-config-airbnb": "^13.0.0",
+    "eslint-config-airbnb-base": "^11.1.0",
-    "eslint-plugin-import": "^2.0.0",
+    "eslint-plugin-import": "^2.2.0",
-    "eslint-plugin-jsx-a11y": "^2.2.3",
-    "eslint-plugin-react": "^6.8.0",
     "grunt": "~1.0.0",
     "grunt-contrib-watch": "^1.0.0",
     "istanbul": "^0.4.2",

public/src/modules/helpers.js

@@ -1,7 +1,6 @@
-(function (exports) {
 'use strict';
 
-
+(function (exports) {
 	// export the class if we are in a Node-like system.
 	if (typeof module === 'object' && module.exports === exports) {
 		exports = module.exports/* = SemVer*/;

public/src/modules/settings/array.js

@@ -1,3 +1,5 @@
+'use strict';
+
 define('settings/array', function () {
 
 	var Settings = null;

public/src/modules/settings/checkbox.js

@@ -1,3 +1,5 @@
+'use strict';
+
 define('settings/checkbox', function () {
 
 	var Settings = null;

public/src/modules/settings/key.js

@@ -1,3 +1,5 @@
+'use strict';
+
 define('settings/key', function () {
 
 	var Settings = null;

public/src/modules/settings/number.js

@@ -1,3 +1,5 @@
+'use strict';
+
 define('settings/number', function () {
 
 	return {

public/src/modules/settings/object.js

@@ -1,3 +1,5 @@
+'use strict';
+
 define('settings/object', function () {
 
 	var Settings = null;

public/src/modules/settings/select.js

@@ -1,3 +1,5 @@
+'use strict';
+
 define('settings/select', function () {
 
 	var Settings = null;

public/src/modules/settings/textarea.js

@@ -1,3 +1,5 @@
+'use strict';
+
 define('settings/textarea', function () {
 
 	var Settings = null;

public/src/modules/translator.js

@@ -1,7 +1,6 @@
-
+'use strict';
 
 (function (factory) {
-	'use strict';
 	function loadClient(language, namespace) {
 		return Promise.resolve(jQuery.getJSON(config.relative_path + '/assets/language/' + language + '/' + namespace + '.json?' + config['cache-buster']));
 	}
@@ -44,7 +43,6 @@
 		window.translator = factory(window.string, loadClient, warn);
 	}
 }(function (string, load, warn) {
-	'use strict';
 	var assign = Object.assign || jQuery.extend;
 	function classCallCheck(instance, Constructor) { if (!(instance instanceof Constructor)) { throw new TypeError('Cannot call a class as a function'); } }
 

public/src/require-config.js

@@ -1,3 +1,5 @@
+'use strict';
+
 require.config({
 	baseUrl: config.relative_path + '/assets/src/modules',
 	waitSeconds: 7,

public/src/utils.js

@@ -1,6 +1,6 @@
-(function (module) {
 'use strict';
 
+(function (module) {
 	var utils;
 	var fs;
 	var XRegExp;

src/posts/cache.js

@@ -1,3 +1,5 @@
+'use strict';
+
 var	LRU = require('lru-cache');
 var meta = require('../meta');
 

src/routes/authentication.js

@@ -1,6 +1,6 @@
-(function (Auth) {
 'use strict';
 
+(function (Auth) {
 	var passport = require('passport');
 	var	passportLocal = require('passport-local').Strategy;
 	var	nconf = require('nconf');

test/mocks/databasemock.js

@@ -1,12 +1,11 @@
+'use strict';
+
 /**
  * Database Mock - wrapper for database.js, makes system use separate test db, instead of production
  * ATTENTION: testing db is flushed before every use!
  */
 
 (function (module) {
-	'use strict';
-
-
 	var async = require('async');
 	var winston = require('winston');
 	var path = require('path');

test/mocks/newXhr.js

@@ -1,3 +1,5 @@
+'use strict';
+
 // see https://gist.github.com/jfromaniello/4087861#gistcomment-1447029
 // XMLHttpRequest to override.