Merge pull request strongloop#12 from strongloop/feature/eslint

Feature/eslint

Author: Amir-61

.eslintignore

@@ -0,0 +1,10 @@
+{
+  "extends": "loopback",
+  "rules": {
+    "max-len": ["error", 80, 4, {
+      "ignoreComments": true,
+      "ignoreUrls": true,
+      "ignorePattern": "^\\s*var\\s.+=\\s*(require\\s*\\()|(/)"
+    }]
+   }
+}

.eslintrc

@@ -4,7 +4,8 @@
   "main": "server/server.js",
   "license": "MIT",
   "scripts": {
-    "pretest": "jshint ."
+    "lint": "eslint .",
+    "posttest": "npm run lint"    
   },
   "dependencies": {
     "async": "~0.9.0",
@@ -17,6 +18,7 @@
     "serve-favicon": "^2.0.1"
   },
   "devDependencies": {
-    "jshint": "^2.8.0"
+    "eslint": "^2.7.0",
+    "eslint-config-loopback": "^1.0.0"
   }
 }

.jshintignore

@@ -11,21 +11,21 @@ var app = module.exports = loopback();
 boot(app, __dirname);
 
 app.start = function(httpOnly) {
-  if(httpOnly === undefined) {
+  if (httpOnly === undefined) {
     httpOnly = process.env.HTTP;
   }
   var server = null;
-  if(!httpOnly) {
+  if (!httpOnly) {
     var options = {
       key: sslConfig.privateKey,
-      cert: sslConfig.certificate
+      cert: sslConfig.certificate,
     };
     server = https.createServer(options, app);
   } else {
     server = http.createServer(app);
   }
   server.listen(app.get('port'), function() {
-    var baseUrl = (httpOnly? 'http://' : 'https://') + app.get('host') + ':' + app.get('port');
+    var baseUrl = (httpOnly ? 'http://' : 'https://') + app.get('host') + ':' + app.get('port');
     app.emit('started', baseUrl);
     console.log('LoopBack server listening @ %s%s', baseUrl, '/');
     if (app.get('loopback-component-explorer')) {

package.json

@@ -1,5 +1,5 @@
-var path = require('path'),
-    fs = require("fs");
+var path = require('path');
+var fs = require('fs');
 
 exports.privateKey = fs.readFileSync(path.join(__dirname, './private/privatekey.pem')).toString();
 exports.certificate = fs.readFileSync(path.join(__dirname, './private/certificate.pem')).toString();

server/server.js

@@ -8,37 +8,37 @@ var db = app.dataSources.db;
 var notes = [
 ];
 
-for(var i=0; i<500; i++) {
-  notes.push({author: 'Author' + i, title: 'Blog ' + i, content: 'Nice content ' + i, created: new Date(), modified: new Date()});
+for (var i = 0; i < 500; i++) {
+  notes.push({ author: 'Author' + i, title: 'Blog ' + i,
+    content: 'Nice content ' + i, created: new Date(), modified: new Date() });
 }
 
 function importData(Model, data, cb) {
-
   // console.log('Importing data for ' + Model.modelName);
-  Model.destroyAll(function (err) {
-    if(err) {
+  Model.destroyAll(function(err) {
+    if (err) {
       cb(err);
       return;
     }
-    async.each(data, function (d, callback) {
+    async.each(data, function(d, callback) {
       Model.create(d, callback);
     }, cb);
   });
 }
 
 async.series(
   [
-    function (cb) {
+    function(cb) {
       db.autoupdate(cb);
     },
-    importData.bind(null, app.models.note, notes)
-  ], function (err, results) {
-    if(err) {
-      console.error(err);
-    } else {
-      console.log('Done');
-    }
-  });
+    importData.bind(null, app.models.note, notes),
+  ], function(err, results) {
+  if (err) {
+    console.error(err);
+  } else {
+    console.log('Done');
+  }
+});
 
 app.start();