Merge pull request #217 from strongloop/eslint-generated-code

Use ESlint in replace of jshint

Author: davidcheung

.eslintignore

@@ -0,0 +1 @@
+apidocs/loopback-core.js

.eslintrc

@@ -0,0 +1,3 @@
+{
+  "extends": "loopback"
+}

.jshintignore

@@ -6,7 +6,7 @@ var loopbackCoreJs = path.resolve(__dirname, 'loopback-core.js');
 try {
   var generator = require('..');
   var loopback = require('loopback');
-} catch(err) {
+} catch (err) {
   if (err.code === 'MODULE_NOT_FOUND' && fs.existsSync(loopbackCoreJs)) {
     console.log('Cannot load the generator, node_modules were not installed.');
     console.log('Ignoring the error since the output file is already there.');

.jshintrc

@@ -20,7 +20,7 @@ module.exports = function(config) {
       { pattern: 'test.e2e/**/*.js', included: false },
 
       // Include lib/ files to let Karma watch for changes there
-      { pattern: 'lib/**/*.js', included: false }
+      { pattern: 'lib/**/*.js', included: false },
     ],
 
     // list of files to exclude
@@ -35,7 +35,7 @@ module.exports = function(config) {
 
     // CI friendly test output
     junitReporter: {
-      outputFile: 'karma-xunit.xml'
+      outputFile: 'karma-xunit.xml',
     },
 
 
@@ -73,6 +73,6 @@ module.exports = function(config) {
 
     // Continuous Integration mode
     // if true, it capture browsers, run tests and exit
-    singleRun: false
+    singleRun: false,
   });
 };

apidocs/describe-builtin-models.js

@@ -2,7 +2,7 @@ var fs = require('fs');
 var ejs = require('ejs');
 
 ejs.filters.q = function(obj) {
-  return JSON.stringify(obj, null, 2 );
+  return JSON.stringify(obj, null, 2);
 };
 
 /**
@@ -38,7 +38,7 @@ module.exports = function generateServices(app, ngModuleName, apiUrl) {
   return ejs.render(servicesTemplate, {
     moduleName: ngModuleName,
     models: models,
-    urlBase: apiUrl.replace(/\/+$/, '')
+    urlBase: apiUrl.replace(/\/+$/, ''),
   });
 };
 
@@ -60,25 +60,25 @@ function describeModels(app) {
     // the sharedCtor parameters should be added to the parameters
     // of prototype methods.
     c.methods.forEach(function fixArgsOfPrototypeMethods(method, key) {
-      if (method.name=='create') {
+      if (method.name == 'create') {
         var createMany = Object.create(method);
-            createMany.name = 'createMany';
-            createMany.isReturningArray = function() { return true; };
-            c.methods.splice(key+1, 0, createMany);
+        createMany.name = 'createMany';
+        createMany.isReturningArray = function() { return true; };
+        c.methods.splice(key + 1, 0, createMany);
       }
       var ctor = method.restClass.ctor;
       if (!ctor || method.sharedMethod.isStatic) return;
       method.accepts = ctor.accepts.concat(method.accepts);
 
-      if(!method.accepts) return;
+      if (!method.accepts) return;
 
       // Any extra http action arguments in the path need to be added to the
       // angular resource actions as params
       method.accepts.forEach(function findResourceParams(arg) {
-        if(!arg.http) return;
+        if (!arg.http) return;
 
-        if(arg.http.source === 'path' && arg.arg !== 'id') {
-          if(!method.resourceParams) {
+        if (arg.http.source === 'path' && arg.arg !== 'id') {
+          if (!method.resourceParams) {
             method.resourceParams = [];
             method.hasResourceParams = true;
           }
@@ -152,7 +152,7 @@ function buildScopeMethod(models, modelName, method) {
 
     modelClass.scopes[scopeName] = {
       methods: {},
-      targetClass: targetClass
+      targetClass: targetClass,
     };
   } else if (modelClass.scopes[scopeName] === null) {
     // Skip the scope, the warning was already reported
@@ -184,18 +184,18 @@ function buildScopeMethod(models, modelName, method) {
 
   var reverseModel = findModelByName(models, targetClass);
   reverseModel.methods.push(reverseMethod);
-  if(reverseMethod.name.match(/create/)){
+  if (reverseMethod.name.match(/create/)) {
     var createMany = Object.create(reverseMethod);
-        createMany.name = createMany.name.replace(
+    createMany.name = createMany.name.replace(
           /create/,
           'createMany'
         );
-        createMany.internal = createMany.internal.replace(
+    createMany.internal = createMany.internal.replace(
           /create/,
           'createMany'
         );
-        createMany.isReturningArray = function() { return true; };
-        reverseModel.methods.push(createMany);
+    createMany.isReturningArray = function() { return true; };
+    reverseModel.methods.push(createMany);
   }
 
   var scopeMethod = Object.create(method);
@@ -204,7 +204,7 @@ function buildScopeMethod(models, modelName, method) {
   scopeMethod.deprecated = false;
   scopeMethod.internal = false;
   modelClass.scopes[scopeName].methods[apiName] = scopeMethod;
-  if(scopeMethod.name.match(/create/)){
+  if (scopeMethod.name.match(/create/)) {
     var scopeCreateMany = Object.create(scopeMethod);
     scopeCreateMany.name = scopeCreateMany.name.replace(
       /create/,