update the style of eslint config with some minor changes

.eslintignore

@@ -0,0 +1,4 @@
+node_modules/**
+build/index.js
+client/**
+tests/**

.eslintrc.js

@@ -0,0 +1,35 @@
+module.exports = {
+ env: {
+ browser: true,
+ node: true,
+ },
+ rules: {
+ // best practices:
+ // encourages use of dot notation whenever possible
+ 'dot-notation': ['error', { allowKeywords: false }],
+
+ // variables:
+ // disallow declaration of variables that are not used in the code
+ 'no-unused-vars': ['error', { vars: 'local', args: 'after-used', ignoreRestSiblings: true }],
+
+ // stylistic issues:
+ // enforce one true brace style
+ 'brace-style': ['error', '1tbs', { allowSingleLine: true }],
+ // require camel case names
+ camelcase: ['error', { properties: 'never' }],
+ // this option sets a specific tab width for your code
+ indent: ['error', 2, { VariableDeclarator: 2, SwitchCase: 1 }],
+ // specify the maximum length of a line in your program
+ 'max-len': ['error', 120, 2],
+ // enforce a maximum depth that callbacks can be nested
+ 'max-nested-callbacks': ['error', 4],
+ // require a capital letter for constructors
+ 'new-cap': ['error', { newIsCap: true, capIsNew: false }],
+ // disallow multiple empty lines and only one newline at the end
+ 'no-multiple-empty-lines': ['error', { max: 1, maxEOF: 1 }],
+ // specify whether double or single quotes should be used
+ quotes: ['error', 'single', 'avoid-escape'],
+ // require or disallow use of quotes around object literal property names
+ 'quote-props': ['error', 'as-needed', { keywords: true }],
+ }
+};

library/modules/es6.date.to-json.js

@@ -8,6 +8,7 @@ var $export = require('./_export')
 $export($export.P + $export.F * require('./_fails')(function(){
  return new Date(NaN).toJSON() !== null || Date.prototype.toJSON.call({toISOString: function(){ return 1; }}) !== 1;
 }), 'Date', {
+ // eslint-disable-next-line no-unused-vars
  toJSON: function toJSON(key){
  var O = toObject(this)
  , pv = toPrimitive(O);

library/modules/es6.promise.js

@@ -193,6 +193,7 @@ if(!USE_NATIVE){
  $reject.call(this, err);
  }
  };
+ // eslint-disable-next-line no-unused-vars
  Internal = function Promise(executor){
  this._c = []; // <- awaiting reactions
  this._a = undefined; // <- checked in isUnhandled reactions

modules/es6.date.to-json.js

@@ -6,6 +6,7 @@ var $export = require('./_export')
 $export($export.P + $export.F * require('./_fails')(function(){
  return new Date(NaN).toJSON() !== null || Date.prototype.toJSON.call({toISOString: function(){ return 1; }}) !== 1;
 }), 'Date', {
+ // eslint-disable-next-line no-unused-vars
  toJSON: function toJSON(key){
  var O = toObject(this)
  , pv = toPrimitive(O);

modules/es6.promise.js

@@ -193,6 +193,7 @@ if(!USE_NATIVE){
  $reject.call(this, err);
  }
  };
+ // eslint-disable-next-line no-unused-vars
  Internal = function Promise(executor){
  this._c = []; // <- awaiting reactions
  this._a = undefined; // <- checked in isUnhandled reactions

modules/library/es6.date.to-json.js

@@ -8,6 +8,7 @@ var $export = require('./_export')
 $export($export.P + $export.F * require('./_fails')(function(){
  return new Date(NaN).toJSON() !== null || Date.prototype.toJSON.call({toISOString: function(){ return 1; }}) !== 1;
 }), 'Date', {
+ // eslint-disable-next-line no-unused-vars
  toJSON: function toJSON(key){
  var O = toObject(this)
  , pv = toPrimitive(O);

package.json

@@ -33,7 +33,7 @@
  },
  "scripts": {
  "grunt": "grunt",
- "lint": "eslint es5 es6 es7 stage web core fn modules",
+ "lint": "eslint ./",
  "promises-tests": "promises-aplus-tests tests/promises-aplus/adapter",
  "observables-tests": "node tests/observables/adapter && node tests/observables/adapter-library",
  "test": "npm run lint && npm run grunt livescript client karma:default && npm run grunt clean copy library karma:library && npm run promises-tests && npm run observables-tests && lsc tests/commonjs"