Adds eslint on src and src/__tests__ (#638)

* Adds linting to tests

* Adds linting to sources

* Adds linting

* Adds linting for integration spec

* Updates docs with options removed where necessary

Author: flovilmart

.eslintignore

@@ -0,0 +1 @@
+node_modules

.eslintrc.json

@@ -0,0 +1,27 @@
+{
+    "root": true,
+    "extends": "eslint:recommended",
+    "env": {
+        "node": true,
+        "es6": true
+    },
+    "parser": "babel-eslint",
+    "plugins": [
+        "flowtype"
+    ],
+    "parserOptions": {
+        "ecmaVersion": 6,
+        "sourceType": "module"
+    },
+    "rules": {
+        "indent": ["error", 2],
+        "linebreak-style": ["error", "unix"],
+        "no-trailing-spaces": 2,
+        "eol-last": 2,
+        "space-in-parens": ["error", "never"],
+        "no-multiple-empty-lines": 1,
+        "prefer-const": "error",
+        "space-infix-ops": "error",
+        "no-useless-escape": "off"
+    }
+}

.gitignore

@@ -12,3 +12,5 @@ integration/test_logs
 test_logs
 out/
 docs/
+.eslintcache
+

.travis.yml

@@ -23,6 +23,7 @@ jobs:
       - npm install -g mongodb-runner
       - mongodb-runner start
       script: 
+      - npm run lint
       - npm test -- --maxWorkers=4
       - npm run integration
       after_script: ./node_modules/codecov/bin/codecov -f ./coverage/coverage-final.json && rm -rf ./coverage

2.0.0.md

@@ -14,19 +14,19 @@ With native promises, `.done` and `.fail` don't exist and are replaces by `.then
 // before
 const query = new Parse.Query();
 query.find()
-.done((results) => {
+.done((results) => {
 
 })
-.fail((error) => {
+.fail((error) => {
 
 });
 
 // after
 query.find()
-.then((results) => {
+.then((results) => {
 
 })
-.catch((error) => {
+.catch((error) => {
 
 });
 ```
@@ -37,13 +37,13 @@ query.find()
 // before
 const query = new Parse.Query();
 query.find()
-.always((result) => {
+.always((result) => {
 
 });
 
 // after
 query.find()
-.finally((result) => {
+.finally((result) => {
 
 });
 ```
@@ -55,15 +55,15 @@ With native promises, the constructor is different, you will need to update to t
 ```js
 // before
 const promise = new Promise();
-doSomethingAsync((error, success) => {
+doSomethingAsync((error, success) => {
   if (error) { promise.reject(error); }
   else { promise.resolve(success); }
 });
 return promise;
 
 // after
-return new Promise((resolve, reject) => {
-  doSomethingAsync((error, success) => {
+return new Promise((resolve, reject) => {
+  doSomethingAsync((error, success) => {
     if (error) { reject(error); }
     else { resolve(success); }
   });

integration/cloud/main.js

@@ -1,3 +1,4 @@
+/* global Parse */
 Parse.Cloud.define("bar", function(request, response) {
   if (request.params.key2 === "value1") {
     response.success('Foo');
@@ -22,4 +23,4 @@ Parse.Cloud.job('CloudJob2', function(request, response) {
 
 Parse.Cloud.job('CloudJobFailing', function(request, response) {
   response.error('cloud job failed');
-});
+});

integration/test/.eslintrc.json

@@ -0,0 +1,10 @@
+{
+  "env": {
+      "jasmine": true
+  },
+  "globals": {},
+  "rules": {
+    "no-console": [0],
+    "no-var": "error"
+  }
+}