added javascript styling rules (#2748)

* added javascript styling rules

* use nodejs 4.4.7 instead of 0.10.x

Author: Chlodochar

.eslintignore

@@ -0,0 +1,6 @@
+**/vendor/**
+**/dist/**
+**/node_modules/**
+**/server.js
+**/venv/**
+Gruntfile.js

.eslintrc.json

@@ -0,0 +1,22 @@
+{
+  "extends": "standard",
+  "parserOptions": {
+    "ecmaFeatures": {
+      "jsx": false
+    },
+    "ecmaVersion": 6,
+    "sourceType": "module"
+  },
+  "env": {
+    "browser": true,
+    "jquery": true
+  },
+  "globals": {
+    "google": true,
+    "centerLat": true,
+    "centerLng": true
+  },
+  "rules": {
+    "semi": 0
+  }
+}

.travis.yml

@@ -5,11 +5,11 @@ python:
 env:
   - FLASK=0.11.1
 install:
+  - nvm install 4.4.7
   - pip install Flask==$FLASK
   - pip install -r requirements.txt
   - npm install grunt-cli -g
   - npm install
 script:
-  - npm run lint
   - npm run build
   - nosetests

Gruntfile.js

@@ -15,17 +15,8 @@ module.exports = function(grunt) {
         }
       }
     },
-    jshint: {
-      files: ['Gruntfile.js', 'js/*.js', '!js/vendor/**/*.js'],
-      options: {
-        reporter: require('jshint-stylish'),
-        globals: {
-          jQuery: true,
-          console: true,
-          module: true,
-          document: true
-        }
-      }
+    eslint: {
+      src: ['static/js/*.js', '!js/vendor/**/*.js']
     },
     babel: {
       options: {
@@ -114,7 +105,7 @@ module.exports = function(grunt) {
   });
 
   grunt.loadNpmTasks('grunt-contrib-uglify');
-  grunt.loadNpmTasks('grunt-contrib-jshint');
+  grunt.loadNpmTasks('grunt-eslint');
   grunt.loadNpmTasks('grunt-contrib-watch');
   grunt.loadNpmTasks('grunt-sass');
   grunt.loadNpmTasks('grunt-contrib-copy');
@@ -128,7 +119,7 @@ module.exports = function(grunt) {
 
   grunt.registerTask('js-build', ['babel', 'uglify', 'minjson']);
   grunt.registerTask('css-build', ['sass', 'cssmin']);
-  grunt.registerTask('js-lint', ['jshint']);
+  grunt.registerTask('js-lint', ['eslint']);
 
   grunt.registerTask('build', ['clean', 'js-build', 'css-build']);
   grunt.registerTask('lint', ['js-lint']);

package.json

@@ -5,6 +5,10 @@
   "main": "Gruntfile.js",
   "dependencies": {
     "babel-preset-es2015": "^6.9.0",
+    "eslint": "^3.1.1",
+    "eslint-config-standard": "^5.3.5",
+    "eslint-plugin-promise": "^2.0.0",
+    "eslint-plugin-standard": "^2.0.0",
     "grunt": "^0.4.5",
     "grunt-babel": "^6.0.0",
     "grunt-cli": "^1.2.0",
@@ -13,17 +17,16 @@
     "grunt-contrib-connect": "latest",
     "grunt-contrib-copy": "latest",
     "grunt-contrib-cssmin": "latest",
-    "grunt-contrib-jshint": "latest",
     "grunt-contrib-uglify": "latest",
     "grunt-contrib-watch": "latest",
+    "grunt-eslint": "^19.0.0",
     "grunt-html-validation": "^0.1.18",
     "grunt-include-file": "^0.1.1",
     "grunt-legacssy": "latest",
     "grunt-minjson": "latest",
     "grunt-sass": "latest",
     "grunt-unused": "latest",
-    "grunt-usemin": "latest",
-    "jshint-stylish": "latest"
+    "grunt-usemin": "latest"
   },
   "scripts": {
     "postinstall": "./node_modules/.bin/grunt build",