Merge pull request #450 from elboletaire/f/eslint

Replace jshint by eslint

.eslintignore

@@ -0,0 +1,14 @@
+node_modules
+examples/node_modules
+examples/**/_package/
+*.min.js
+client/src/renderers/support/jade-runtime.js
+src/components/oc-client/src/head.load.js
+src/components/oc-client/_package/**/*
+src/components/base-component-handlebars/_package/**/*
+src/components/base-component-handlebars/server.js
+src/components/base-component-jade/_package/**/*
+src/components/base-component-jade/server.js
+test/fixtures
+test/front-end
+test/configuration

.eslintrc.json

@@ -0,0 +1,271 @@
+{
+  "env": {
+    "browser": true,
+    "es6": true,
+    "node": true
+  },
+  "extends": "eslint:recommended",
+  "parserOptions": {
+    "sourceType": "module"
+  },
+  "rules": {
+    "accessor-pairs": "error",
+    "array-bracket-spacing": "off",
+    "array-callback-return": "off",
+    "arrow-body-style": "error",
+    "arrow-parens": "off",
+    "arrow-spacing": [
+      "error",
+      {
+        "after": true,
+        "before": true
+      }
+    ],
+    "block-scoped-var": "off",
+    "block-spacing": "off",
+    "brace-style": "off",
+    "callback-return": "off",
+    "camelcase": "off",
+    "capitalized-comments": "off",
+    "class-methods-use-this": "error",
+    "comma-dangle": "off",
+    "comma-spacing": "off",
+    "comma-style": "off",
+    "complexity": "off",
+    "computed-property-spacing": "off",
+    "consistent-return": "off",
+    "consistent-this": "off",
+    "curly": "off",
+    "default-case": "error",
+    "dot-location": [
+      "error",
+      "property"
+    ],
+    "dot-notation": "off",
+    "eol-last": "off",
+    "eqeqeq": "off",
+    "func-call-spacing": "off",
+    "func-name-matching": "error",
+    "func-names": "off",
+    "func-style": "off",
+    "generator-star-spacing": "error",
+    "global-require": "off",
+    "guard-for-in": "off",
+    "handle-callback-err": "off",
+    "id-blacklist": "error",
+    "id-length": "off",
+    "id-match": "error",
+    "indent": "off",
+    "init-declarations": "off",
+    "jsx-quotes": "error",
+    "key-spacing": "off",
+    "keyword-spacing": "off",
+    "line-comment-position": "off",
+    "linebreak-style": "off",
+    "lines-around-comment": "off",
+    "lines-around-directive": "off",
+    "max-depth": "off",
+    "max-len": "off",
+    "max-lines": "off",
+    "max-nested-callbacks": "error",
+    "max-params": "off",
+    "max-statements": "off",
+    "max-statements-per-line": "off",
+    "multiline-ternary": "off",
+    "new-parens": "error",
+    "newline-after-var": "off",
+    "newline-before-return": "off",
+    "newline-per-chained-call": "off",
+    "no-alert": "error",
+    "no-array-constructor": "error",
+    "no-await-in-loop": "error",
+    "no-bitwise": "off",
+    "no-caller": "error",
+    "no-catch-shadow": "off",
+    "no-compare-neg-zero": "error",
+    "no-confusing-arrow": "error",
+    "no-continue": "off",
+    "no-console": "warn",
+    "no-div-regex": "error",
+    "no-duplicate-imports": "error",
+    "no-else-return": "off",
+    "no-empty": [
+      "error",
+      {
+        "allowEmptyCatch": true
+      }
+    ],
+    "no-empty-function": "off",
+    "no-eq-null": "off",
+    "no-eval": "off",
+    "no-extend-native": "error",
+    "no-extra-bind": "error",
+    "no-extra-label": "error",
+    "no-extra-parens": "off",
+    "no-floating-decimal": "error",
+    "no-extra-boolean-cast": "warn",
+    "no-implicit-coercion": [
+      "error",
+      {
+        "boolean": false,
+        "number": false,
+        "string": false
+      }
+    ],
+    "no-implicit-globals": "error",
+    "no-implied-eval": "error",
+    "no-inline-comments": "off",
+    "no-inner-declarations": [
+      "error",
+      "functions"
+    ],
+    "no-invalid-this": "off",
+    "no-iterator": "error",
+    "no-label-var": "error",
+    "no-labels": "error",
+    "no-lone-blocks": "error",
+    "no-lonely-if": "off",
+    "no-loop-func": "error",
+    "no-magic-numbers": "off",
+    "no-mixed-operators": "off",
+    "no-mixed-requires": "off",
+    "no-multi-assign": "off",
+    "no-multi-spaces": "off",
+    "no-multi-str": "error",
+    "no-multiple-empty-lines": "off",
+    "no-native-reassign": "error",
+    "no-negated-condition": "off",
+    "no-negated-in-lhs": "error",
+    "no-nested-ternary": "off",
+    "no-new": "error",
+    "no-new-func": "off",
+    "no-new-object": "error",
+    "no-new-require": "error",
+    "no-new-wrappers": "error",
+    "no-octal-escape": "error",
+    "no-param-reassign": "off",
+    "no-path-concat": "error",
+    "no-plusplus": "off",
+    "no-process-env": "off",
+    "no-process-exit": "off",
+    "no-proto": "error",
+    "no-prototype-builtins": "off",
+    "no-restricted-globals": "error",
+    "no-restricted-imports": "error",
+    "no-restricted-modules": "error",
+    "no-restricted-properties": "error",
+    "no-restricted-syntax": "error",
+    "no-return-assign": "off",
+    "no-return-await": "error",
+    "no-script-url": "error",
+    "no-self-compare": "off",
+    "no-sequences": "off",
+    "no-shadow": "off",
+    "no-shadow-restricted-names": "off",
+    "no-spaced-func": "off",
+    "no-sync": "off",
+    "no-tabs": "off",
+    "no-template-curly-in-string": "off",
+    "no-ternary": "off",
+    "no-throw-literal": "off",
+    "no-trailing-spaces": "off",
+    "no-undef-init": "off",
+    "no-undefined": "off",
+    "no-undef": "off",
+    "no-underscore-dangle": "off",
+    "no-unmodified-loop-condition": "off",
+    "no-unneeded-ternary": "off",
+    "no-unused-expressions": "off",
+    "no-unused-vars": "error",
+    "no-use-before-define": "off",
+    "no-useless-call": "off",
+    "no-useless-computed-key": "error",
+    "no-useless-concat": "off",
+    "no-useless-constructor": "error",
+    "no-useless-escape": "off",
+    "no-useless-rename": "error",
+    "no-useless-return": "error",
+    "no-var": "off",
+    "no-void": "off",
+    "no-warning-comments": "error",
+    "no-whitespace-before-property": "error",
+    "no-with": "error",
+    "nonblock-statement-body-position": [
+      "error",
+      "any"
+    ],
+    "object-curly-newline": "off",
+    "object-curly-spacing": "off",
+    "object-property-newline": "off",
+    "object-shorthand": "off",
+    "one-var": "off",
+    "one-var-declaration-per-line": "off",
+    "operator-assignment": "off",
+    "operator-linebreak": "off",
+    "padded-blocks": "off",
+    "prefer-arrow-callback": "off",
+    "prefer-const": "off",
+    "prefer-destructuring": [
+      "error",
+      {
+        "array": false,
+        "object": false
+      }
+    ],
+    "prefer-numeric-literals": "error",
+    "prefer-promise-reject-errors": "error",
+    "prefer-reflect": "off",
+    "prefer-rest-params": "off",
+    "prefer-spread": "off",
+    "prefer-template": "off",
+    "quote-props": "off",
+    "quotes": "off",
+    "radix": [
+      "error",
+      "always"
+    ],
+    "require-await": "error",
+    "require-jsdoc": "off",
+    "rest-spread-spacing": "error",
+    "semi": ["error", "always"],
+    "semi-spacing": "off",
+    "sort-imports": "error",
+    "sort-keys": "off",
+    "sort-vars": "off",
+    "space-before-blocks": "off",
+    "space-before-function-paren": "off",
+    "space-in-parens": "off",
+    "space-infix-ops": "off",
+    "space-unary-ops": [
+      "error",
+      {
+        "nonwords": false,
+        "words": false
+      }
+    ],
+    "spaced-comment": "off",
+    "strict": "off",
+    "symbol-description": "error",
+    "template-curly-spacing": [
+      "error",
+      "never"
+    ],
+    "template-tag-spacing": "error",
+    "unicode-bom": [
+      "error",
+      "never"
+    ],
+    "valid-jsdoc": "off",
+    "valid-typeof": [
+      "error",
+      {
+        "requireStringLiterals": false
+      }
+    ],
+    "vars-on-top": "off",
+    "wrap-iife": "off",
+    "wrap-regex": "off",
+    "yield-star-spacing": "error",
+    "yoda": "off"
+  }
+}

Gruntfile.js

@@ -23,9 +23,9 @@ module.exports = function(grunt){
 
   grunt.registerTask('default', ['test-local', 'build']);
   grunt.registerTask('sauce', ['karma:sauce-linux', 'karma:sauce-osx', 'karma:sauce-windows']);
-  grunt.registerTask('test-local', ['jshint:all', 'mochaTest:unit', 'mochaTest:acceptance', 'karma:local']);
-  grunt.registerTask('test-local-silent', ['jshint:all', 'mochaTest:silent', 'karma:local']);
-  grunt.registerTask('test', ['jshint:all', 'mochaTest:unit', 'mochaTest:integration', 'mochaTest:acceptance']);
+  grunt.registerTask('test-local', ['eslint', 'mochaTest:unit', 'mochaTest:acceptance', 'karma:local']);
+  grunt.registerTask('test-local-silent', ['eslint', 'mochaTest:silent', 'karma:local']);
+  grunt.registerTask('test', ['eslint', 'mochaTest:unit', 'mochaTest:integration', 'mochaTest:acceptance']);
   grunt.registerTask('git-stage', [
     'gitadd:versionFiles',
     'gitcommit:version',

package.json

@@ -31,7 +31,7 @@
     "chai": "1.10.0",
     "cheerio": "0.22.0",
     "grunt": "^1.0.1",
-    "grunt-contrib-jshint": "^1.0.0",
+    "grunt-eslint": "^19.0.0",
     "grunt-git": "^1.0.0",
     "grunt-karma": "^2.0.0",
     "grunt-mocha-test": "^0.12.7",

src/cli/domain/package-template.js

@@ -43,15 +43,14 @@ var compileView = function(viewPath, type, cb) {
 module.exports = function(params, callback){
 
   var viewSrc = params.ocOptions.files.template.src,
-      viewPath = path.join(params.componentPath, viewSrc),
-      compiled;
+      viewPath = path.join(params.componentPath, viewSrc);
 
   if(!fs.existsSync(viewPath)){
     return callback(format(strings.errors.cli.TEMPLATE_NOT_FOUND, viewSrc));
   }
 
   try {
-    compiled = compileView(viewPath, params.ocOptions.files.template.type, (err, compiled) => {
+    compileView(viewPath, params.ocOptions.files.template.type, (err, compiled) => {
       if (err) {
         return callback(format('{0} compilation failed - {1}', viewSrc, err));
       }

src/registry/routes/helpers/apply-default-values.js

@@ -8,7 +8,7 @@ module.exports = function(requestParameters, expectedParameters) {
   });
 
   _.forEach(optionalParametersWithDefaults, function(expectedParameter, expectedParameterName){
-  	var param = requestParameters[expectedParameterName];
+    var param = requestParameters[expectedParameterName];
     if(_.isUndefined(param) || _.isNull(param)) {
       requestParameters[expectedParameterName] = expectedParameter.default;
     }

tasks/eslint.js

@@ -0,0 +1,5 @@
+'use strict';
+
+module.exports = {
+  target: ['.']
+};