Merge pull request #1179 from strongloop/update/eslint

Upgrade eslint & config to latest

Author: bajtos

examples/datasource-app.js

@@ -107,7 +107,7 @@ Article.create(function(e, article) {
 // should be able to attach a data source to an existing model
 var modelBuilder = new ModelBuilder();
 
-Color = modelBuilder.define('Color', {
+const Color = modelBuilder.define('Color', {
   name: String,
 });
 

lib/connectors/memory.js

@@ -5,6 +5,7 @@
 
 'use strict';
 
+/* global window:false */
 var g = require('strong-globalize')();
 var util = require('util');
 var Connector = require('loopback-connector').Connector;
@@ -823,7 +824,7 @@ Memory.prototype.replaceOrCreate = function(model, data, options, callback) {
     // Calling _createSync to update the collection in a sync way and
     // to guarantee to create it in the same turn of even loop
     return self._createSync(model, data, function(err, id) {
-      if (err) return process.nextTick(function() { cb(err); });
+      if (err) return process.nextTick(function() { callback(err); });
       self.saveToFile(id, function(err, id) {
         self.setIdValue(model, data, id);
         callback(err, self.fromDb(model, data), {isNewInstance: true});

package.json

@@ -33,8 +33,8 @@
   },
   "devDependencies": {
     "async-iterators": "^0.2.2",
-    "eslint": "^2.13.1",
-    "eslint-config-loopback": "^4.0.0",
+    "eslint": "^3.11.1",
+    "eslint-config-loopback": "^6.0.0",
     "mocha": "^2.1.0",
     "should": "^8.0.2"
   },

test/basic-querying.test.js

@@ -6,6 +6,7 @@
 // This test written in mocha+should.js
 'use strict';
 
+/* global getSchema:false, connectorCapabilities:false */
 var should = require('./init.js');
 var async = require('async');
 var db, User;

test/crud-with-options.test.js

@@ -6,6 +6,7 @@
 // This test written in mocha+should.js
 'use strict';
 
+/* global getSchema:false */
 var should = require('./init.js');
 var async = require('async');
 var db, User, options, filter;

test/datatype.test.js

@@ -6,6 +6,7 @@
 // This test written in mocha+should.js
 'use strict';
 
+/* global getSchema:false */
 var should = require('./init.js');
 
 var db, Model;

test/default-scope.test.js

@@ -6,6 +6,7 @@
 // This test written in mocha+should.js
 'use strict';
 
+/* global getSchema:false */
 var should = require('./init.js');
 var async = require('async');
 

test/defaults.test.js

@@ -6,6 +6,7 @@
 // This test written in mocha+should.js
 'use strict';
 
+/* global getSchema:false */
 var should = require('./init.js');
 
 var db = getSchema();

test/hooks.test.js

@@ -6,6 +6,7 @@
 'use strict';
 
 // This test written in mocha+should.js
+/* global getSchema:false */
 var should = require('./init.js');
 
 var j = require('../'),

test/include.test.js

@@ -5,6 +5,7 @@
 
 'use strict';
 
+/* global getSchema:false */
 var should = require('./init.js');
 var async = require('async');
 var assert = require('assert');