Update eslint & config to latest

Signed-off-by: Miroslav Bajtoš <[email protected]>

Author: bajtos

Diff for: example/before-after.js

@@ -5,12 +5,12 @@
 
 'use strict';
 
-var g = require('strong-globalize')();
+const g = require('strong-globalize')();
 // create a set of shared classes
-var remotes = require('../').create();
+const remotes = require('../').create();
 
 // expose a simple object
-var user = remotes.exports.user = {
+const user = remotes.exports.user = {
   greet: function(fn) {
     fn(null, 'hello, world!');
   },
@@ -62,7 +62,7 @@ remotes.before('user.*', function(ctx, next) {
 
 // do something before a dog instance method
 remotes.before('dog.prototype.*', function(ctx, next) {
-  var dog = this;
+  const dog = this;
   g.log('calling a method on %s', dog.name);
   next();
 });

Diff for: example/documentation/index.js

@@ -5,13 +5,13 @@
 
 'use strict';
 
-var g = require('strong-globalize')();
-var http = require('http');
-var remotes = require('./remotes');
-var meta = require('../../ext/meta');
-var swagger = require('../../ext/swagger');
-var port = process.argv[2] || 3000;
-var handler, adapter;
+const g = require('strong-globalize')();
+const http = require('http');
+const remotes = require('./remotes');
+const meta = require('../../ext/meta');
+const swagger = require('../../ext/swagger');
+const port = process.argv[2] || 3000;
+let handler, adapter;
 
 // The installation order sets which routes are captured by Swagger.
 swagger(remotes, {

Diff for: example/documentation/remotes/contract-class.js

@@ -6,13 +6,12 @@
 'use strict';
 
 // This example shows using the helper for a type in a "definitive" fashion.
-var helper = require('../../../').extend(module.exports);
-var clshelper;
+const helper = require('../../../').extend(module.exports);
 
 /**
  * A simple class that contains a name, this time with a custom HTTP contract.
  */
-clshelper = helper.type(ContractClass, {
+const clshelper = helper.type(ContractClass, {
   accepts: [{name: 'name', type: 'string', required: true}],
   http: {path: '/:name'},
 });

Diff for: example/documentation/remotes/contract.js

@@ -5,7 +5,7 @@
 
 'use strict';
 
-var helper = require('../../../').extend(module.exports);
+const helper = require('../../../').extend(module.exports);
 
 /**
  * Returns a secret message.

Diff for: example/documentation/remotes/index.js

@@ -5,7 +5,7 @@
 
 'use strict';
 
-var remotes = require('../../../').create();
+const remotes = require('../../../').create();
 
 /**
  * Example API

Diff for: example/documentation/remotes/simple-class.js

@@ -6,7 +6,7 @@
 'use strict';
 
 // This example shows using the helper for a type in a "post-definition" style.
-var helper = require('../../../').extend(module.exports);
+const helper = require('../../../').extend(module.exports);
 
 /**
  * A simple class that contains a name.

Diff for: example/documentation/remotes/simple.js

@@ -8,7 +8,7 @@
 // This helper adds methods to a module that we assume will be added to the remotes.
 // TODO(schoon) - Make this _the_ API, not a "helper".
 // TODO(schoon) - Document EVERYTHING
-var helper = require('../../../').extend(module.exports);
+const helper = require('../../../').extend(module.exports);
 
 /**
  * Returns a secret message.

Diff for: example/remote-fs.js

@@ -6,10 +6,10 @@
 'use strict';
 
 // create a set of shared classes
-var remotes = require('../').create();
+const remotes = require('../').create();
 
 // share some fs module code
-var fs = remotes.exports.fs = require('fs');
+const fs = remotes.exports.fs = require('fs');
 
 // specifically the createReadStream function
 fs.createReadStream.shared = true;

Diff for: example/root.js

@@ -6,10 +6,10 @@
 'use strict';
 
 // create a set of shared classes
-var remotes = require('../').create();
+const remotes = require('../').create();
 
 // expose a simple object
-var products = remotes.exports.products = {
+const products = remotes.exports.products = {
   find: function(fn) {
     fn(null, ['tv', 'vcr', 'radio']);
   },

Diff for: example/shared-class.js

@@ -5,7 +5,7 @@
 
 'use strict';
 
-var express = require('express');
+const express = require('express');
 
 // define a vanilla JavaScript class
 function Dog(name) {
@@ -33,12 +33,12 @@ Dog.prototype.speak.returns = {arg: 'result', type: 'string', root: true};
 Dog.prototype.speak.shared = true;
 
 // create a set of shared classes
-var remotes = require('../').create();
+const remotes = require('../').create();
 
 // expose the Dog class
 remotes.exports.dog = Dog;
 
-var app = express();
+const app = express();
 app.use(remotes.handler('rest'));
 
 app.listen(3000);

Diff for: example/simple-types.js

@@ -6,10 +6,10 @@
 'use strict';
 
 // create a set of shared classes
-var remotes = require('../').create();
+const remotes = require('../').create();
 
 // expose a simple object
-var user = remotes.exports.user = {
+const user = remotes.exports.user = {
   greet: function(fn) {
     fn(null, {msg: 'hello, world!'});
   },

Diff for: example/simple.js

@@ -6,22 +6,22 @@
 'use strict';
 
 // create a set of shared classes
-var remoting = require('../');
-var SharedClass = remoting.SharedClass;
-var remotes = remoting.create();
-var express = require('express');
-var app = express();
+const remoting = require('../');
+const SharedClass = remoting.SharedClass;
+const remotes = remoting.create();
+const express = require('express');
+const app = express();
 
 // define a class-like object (or constructor)
-var user = {
+const user = {
   greet: function(fn) {
     fn(null, 'hello, world!');
   },
 };
 
 // create a shared class to allow strong-remoting to map
 // http requests to method invocations on your class
-var userSharedClass = new SharedClass('user', user);
+const userSharedClass = new SharedClass('user', user);
 
 // tell strong-remoting about your greet method
 userSharedClass.defineMethod('greet', {

Diff for: example/socket-io/client.js

@@ -5,11 +5,11 @@
 
 'use strict';
 
-var g = require('strong-globalize')();
+const g = require('strong-globalize')();
 
-var Remotes = require('../../client/js/client');
-var SocketIOAdapter = require('../../client/js/socket-io-adapter');
-var remotes = Remotes.connect('http://localhost:3000', SocketIOAdapter);
+const Remotes = require('../../client/js/client');
+const SocketIOAdapter = require('../../client/js/socket-io-adapter');
+const remotes = Remotes.connect('http://localhost:3000', SocketIOAdapter);
 
 remotes.invoke('fs.readFile', {path: 'test.txt'}, function(err, data) {
   g.log(data.toString());

Diff for: example/socket-io/server.js

@@ -6,10 +6,10 @@
 'use strict';
 
 // create a set of shared classes
-var remotes = require('../../').create();
+const remotes = require('../../').create();
 
 // share some fs module code
-var fs = remotes.exports.fs = require('fs');
+const fs = remotes.exports.fs = require('fs');
 
 // specifically the readFile function
 fs.readFile.shared = true;
@@ -21,8 +21,8 @@ fs.readFile.accepts = {arg: 'path', type: 'string'};
 fs.readFile.returns = {arg: 'data', type: 'buffer'};
 
 // event emitter
-var EventEmitter = require('events').EventEmitter;
-var ee = remotes.exports.ee = new EventEmitter();
+const EventEmitter = require('events').EventEmitter;
+const ee = remotes.exports.ee = new EventEmitter();
 
 // expose the on method
 ee.on.shared = true;
@@ -35,7 +35,7 @@ setInterval(function() {
 }, 1000);
 
 // expose it over http
-var server =
+const server =
 require('http')
   .createServer()
   .listen(3000);

Diff for: example/streams.js

@@ -6,15 +6,15 @@
 'use strict';
 
 // faux remote stream
-var destination = process.stdout;
-var fs = require('fs');
-var path = require('path');
+const destination = process.stdout;
+const fs = require('fs');
+const path = require('path');
 
 // create a set of shared classes
-var remotes = require('../').create();
+const remotes = require('../').create();
 
 // our modules
-var fileService = remotes.exports.files = {
+const fileService = remotes.exports.files = {
   upload: function() {
     return destination;
   },

Diff for: ext/meta.js

@@ -13,19 +13,19 @@ module.exports = Meta;
 /**
  * Module dependencies.
  */
-var Remoting = require('../');
+const Remoting = require('../');
 
 /**
  * Create a remotable Meta module for plugging into `RemoteObjects`.
  */
 function Meta(remotes, options) {
   // Unfold options.
-  var name = (options && options.name) || 'meta';
+  const name = (options && options.name) || 'meta';
 
   // We need a temporary REST adapter to discover our available routes.
-  var adapter = remotes.handler('rest').adapter;
-  var extension = {};
-  var helper = Remoting.extend(extension);
+  const adapter = remotes.handler('rest').adapter;
+  const extension = {};
+  const helper = Remoting.extend(extension);
 
   helper.method(routes, {returns: {type: 'object', root: true}});
   function routes(callback) {

Diff for: index.js

@@ -9,7 +9,7 @@
  * remotes ~ public api
  */
 
-var SG = require('strong-globalize');
+const SG = require('strong-globalize');
 SG.SetRootDir(__dirname);
 
 module.exports = require('./lib/remote-objects');

Diff for: lib/context-base.js

@@ -5,10 +5,10 @@
 
 'use strict';
 
-var assert = require('assert');
-var EventEmitter = require('events').EventEmitter;
-var TypeRegistry = require('./type-registry');
-var inherits = require('util').inherits;
+const assert = require('assert');
+const EventEmitter = require('events').EventEmitter;
+const TypeRegistry = require('./type-registry');
+const inherits = require('util').inherits;
 
 module.exports = ContextBase;
 
@@ -34,7 +34,7 @@ inherits(ContextBase, EventEmitter);
 ContextBase.prototype.getScope = function() {
   // Static methods are invoked on the constructor (this = constructor fn)
   // Prototype methods are invoked on the instance (this = instance)
-  var method = this.method;
+  const method = this.method;
   return this.instance ||
     method.ctor ||
     method.sharedMethod && method.sharedMethod.ctor;

Diff for: lib/exports-helper.js

@@ -13,12 +13,12 @@ module.exports = ExportsHelper;
 /*!
  * Module dependencies.
  */
-var debug = require('debug')('strong-remoting:exports-helper');
+const debug = require('debug')('strong-remoting:exports-helper');
 
 /*!
  * Constants
  */
-var PASSTHROUGH_OPTIONS = ['http', 'description', 'notes'];
+const PASSTHROUGH_OPTIONS = ['http', 'description', 'notes'];
 
 /**
  * @class A wrapper to make manipulating the exports object easier.
@@ -40,10 +40,10 @@ function ExportsHelper(obj) {
  */
 ExportsHelper.prototype.setPath = setPath;
 function setPath(path, value) {
-  var self = this;
-  var obj = self._obj;
-  var split = path.split('.');
-  var name = split.pop();
+  const self = this;
+  let obj = self._obj;
+  const split = path.split('.');
+  const name = split.pop();
 
   split.forEach(function(key) {
     if (!obj[key]) {
@@ -65,10 +65,10 @@ function setPath(path, value) {
 ExportsHelper.prototype.addType = type;
 ExportsHelper.prototype.type = type;
 function type(fn, options) {
-  var self = this;
-  var path = options.path || options.name || fn.name || null;
-  var sharedCtor = options.sharedCtor || null;
-  var accepts = options.accepts || null;
+  const self = this;
+  const path = options.path || options.name || fn.name || null;
+  let sharedCtor = options.sharedCtor || null;
+  const accepts = options.accepts || null;
 
   if (!path) {
     // TODO: Error.
@@ -80,7 +80,7 @@ function type(fn, options) {
     // constructor". Instead, this is the lazy find/create sl-remoting uses when
     // a prototype method is called. `getInstance`? `findOrCreate`? `load`?
     sharedCtor = function() {
-      var _args = [].slice.call(arguments);
+      const _args = [].slice.call(arguments);
       _args.pop()(null, fn.apply(null, _args));
     };
   }
@@ -114,11 +114,11 @@ function type(fn, options) {
 ExportsHelper.prototype.addMethod = method;
 ExportsHelper.prototype.method = method;
 function method(fn, options) {
-  var self = this;
-  var path = options.path || options.name || fn.name || null;
-  var accepts = options.accepts || null;
-  var returns = options.returns || null;
-  var errors = options.errors || null;
+  const self = this;
+  const path = options.path || options.name || fn.name || null;
+  const accepts = options.accepts || null;
+  const returns = options.returns || null;
+  const errors = options.errors || null;
 
   if (!path) {
     // TODO: Error.