Merge remote-tracking branch 'origin/master' into issue46

Author: mysticatea

README.md

@@ -94,7 +94,7 @@ The `--fix` option on the command line automatically fixes problems reported by
 |:---|:--------|:------------|
 |  | [no-dupe-keys](./docs/rules/no-dupe-keys.md) | Prevents duplication of field names. |
 | :white_check_mark: | [no-parsing-error](./docs/rules/no-parsing-error.md) | disallow parsing errors in `<template>`. |
-|  | [no-reservered-keys](./docs/rules/no-reservered-keys.md) | Prevent overwrite reserved keys. |
+|  | [no-reserved-keys](./docs/rules/no-reserved-keys.md) | Prevent overwrite reserved keys. |
 |  | [no-shared-component-data](./docs/rules/no-shared-component-data.md) | Enforces component's data property to be a function. |
 |  | [no-template-key](./docs/rules/no-template-key.md) | disallow `key` attribute on `<template>`. |
 |  | [require-render-return](./docs/rules/require-render-return.md) | Enforces render function to always return value. |
@@ -128,8 +128,10 @@ The `--fix` option on the command line automatically fixes problems reported by
 | :white_check_mark: | [no-textarea-mustache](./docs/rules/no-textarea-mustache.md) | disallow mustaches in `<textarea>`. |
 |  | [order-in-components](./docs/rules/order-in-components.md) | Keep order of properties in components |
 | :white_check_mark: | [require-component-is](./docs/rules/require-component-is.md) | require `v-bind:is` of `<component>` elements. |
+|  | [require-default-prop](./docs/rules/require-default-prop.md) | Require default value for props |
 |  | [require-prop-types](./docs/rules/require-prop-types.md) | Prop definitions should be detailed |
 | :white_check_mark: | [require-v-for-key](./docs/rules/require-v-for-key.md) | require `v-bind:key` with `v-for` directives. |
+|  | [this-in-template](./docs/rules/this-in-template.md) | enforce usage of `this` in template. |
 
 
 ### Stylistic Issues
@@ -139,7 +141,10 @@ The `--fix` option on the command line automatically fixes problems reported by
 | :wrench: | [attribute-hyphenation](./docs/rules/attribute-hyphenation.md) | Define a style for the props casing in templates. |
 |  | [html-quotes](./docs/rules/html-quotes.md) | enforce quotes style of HTML attributes. |
 | :wrench: | [html-self-closing](./docs/rules/html-self-closing.md) | enforce self-closing style. |
+|  | [max-attributes-per-line](./docs/rules/max-attributes-per-line.md) | Define the number of attributes allows per line |
+| :wrench: | [mustache-interpolation-spacing](./docs/rules/mustache-interpolation-spacing.md) | enforce unified spacing in mustache interpolations. |
 | :wrench: | [name-property-casing](./docs/rules/name-property-casing.md) | Requires specific casing for the name property in Vue components |
+| :wrench: | [no-multi-spaces](./docs/rules/no-multi-spaces.md) | This rule warns about the usage of extra whitespaces between attributes |
 | :wrench: | [v-bind-style](./docs/rules/v-bind-style.md) | enforce `v-bind` directive style. |
 | :wrench: | [v-on-style](./docs/rules/v-on-style.md) | enforce `v-on` directive style. |
 
@@ -172,6 +177,7 @@ The `--fix` option on the command line automatically fixes problems reported by
 | [no-invalid-v-pre](./rules/no-invalid-v-pre.md) | [valid-v-pre](./rules/valid-v-pre.md) |
 | [no-invalid-v-show](./rules/no-invalid-v-show.md) | [valid-v-show](./rules/valid-v-show.md) |
 | [no-invalid-v-text](./rules/no-invalid-v-text.md) | [valid-v-text](./rules/valid-v-text.md) |
+| [no-reservered-keys](./rules/no-reservered-keys.md) | [no-reserved-keys](./rules/no-reserved-keys.md) |
 
 <!--RULES_TABLE_END-->
 

circle.yml

@@ -4,12 +4,12 @@ machine:
 
 dependencies:
   pre:
-    - nvm install 7
+    - nvm install 6
     - nvm install 8
 
 test:
   override:
-    - npm test
+    - nvm use 4 && npm test
     - nvm use 6 && npm test
-    - nvm use 7 && npm test
     - nvm use 8 && npm test
+    - nvm use 8 && npm i [email protected] --no-save && npm run -s test:base

docs/rules/max-attributes-per-line.md

@@ -0,0 +1,122 @@
+# Define the number of attributes allows per line (max-attributes-per-line)
+
+Limits the maximum number of attributes/properties per line to improve readability.
+
+
+## :book: Rule Details
+
+This rule aims to enforce a number of attributes per line in templates.
+It checks all the elements in a template and verifies that the number of attributes per line does not exceed the defined maximum.
+An attribute is considered to be in a new line when there is a line break between two attributes.
+
+There is a configurable number of attributes that are acceptable in one-line case (default 3), as well as how many attributes are acceptable per line in multi-line case (default 1).
+
+:-1: Examples of **incorrect** code for this rule:
+
+```html
+<component lorem="1" ipsum="2" dolor="3" sit="4">
+</component>
+
+<component
+  lorem="1" ipsum="2"
+  dolor="3"
+  sit="4"
+>
+</component>
+```
+
+:+1: Examples of **correct** code for this rule:
+
+```html
+<component lorem="1" ipsum="2" dolor="3">
+</component>
+
+<component
+  lorem="1"
+  ipsum="2"
+  dolor="3"
+  sit="4"
+>
+</component>
+
+```
+
+### :wrench:  Options
+
+```
+{
+  "vue/max-attributes-per-line": [{
+    "singleline": 3,
+    "multiline": {
+      max: 1,
+      allowFirstLine: false
+    }
+  }]
+}
+```
+
+#### `allowFirstLine`
+For multi-line declarations, defines if allows attributes to be put in the first line. (Default false)
+
+:-1: Example of **incorrect** code for this setting:
+```html
+// [{ "multiline": { "allowFirstLine": false }}]
+<component foo="John" bar="Smith"
+  foobar={5555555}>
+</component>;
+```
+
+:+1: Example of **correct** code for this setting:
+```html
+// [{ "multiline": { "allowFirstLine": false }}]
+<component
+  foo="John"
+  bar="Smith"
+  foobar={5555555}
+  >
+</component>;
+```
+
+
+#### `singleline`
+Number of maximum attributes per line when the opening tag is in a single line. (Default is 3)
+
+:-1: Example of **incorrect** code for this setting:
+```html
+// [{"singleline": 2,}]
+<component foo="John" bar="Smith" foobar={5555555}></component>;
+```
+
+:+1: Example of **correct** code for this setting:
+```html
+// [{"singleline": 3,}]
+<component foo="John" bar="Smith" foobar={5555555}></component>;
+```
+
+
+#### `multiline`
+Number of maximum attributes per line when a tag is in multiple lines. (Default is 1)
+
+:-1: Example of **incorrect** code for this setting:
+```html
+// [{"multiline": 1}]
+<component foo="John" bar="Smith"
+  foobar={5555555}>
+</component>;
+```
+
+:+1: Example of **correct** code for this setting:
+```html
+// [{"multiline": 1}]
+<component
+  foo="John"
+  bar="Smith"
+  foobar={5555555}
+  >
+</component>;
+```
+
+## When Not To Use It
+
+If you do not want to check the number of attributes declared per line you can disable this rule.
+

docs/rules/mustache-interpolation-spacing.md

@@ -0,0 +1,67 @@
+# enforce unified spacing in mustache interpolations. (mustache-interpolation-spacing)
+
+- :wrench: The `--fix` option on the [command line](http://eslint.org/docs/user-guide/command-line-interface#fix) can automatically fix some of the problems reported by this rule.
+
+## :book: Rule Details
+
+This rule aims to enforce unified spacing in mustache interpolations.
+
+:-1: Examples of **incorrect** code for this rule:
+
+```html
+<template>
+  <div>{{   text   }}</div>
+</template>
+```
+
+:+1: Examples of **correct** code for this rule:
+
+```html
+<template>
+  <div>{{ text }}</div>
+</template>
+```
+
+## :wrench: Options
+
+Default spacing is set to `always`
+
+```
+'vue/mustache-interpolation-spacing': [2, 'always'|'never']
+```
+
+### `"always"` - Expect one space between expression and curly brackets.
+
+:+1: Examples of **correct** code`:
+
+```html
+<template>
+  <div>{{ text }}</div>
+</template>
+```
+
+:-1: Examples of **incorrect** code`:
+
+```html
+<template>
+  <div>{{text}}</div>
+</template>
+```
+
+### `"never"` - Expect no spaces between expression and curly brackets.
+
+:+1: Examples of **correct** code`:
+
+```html
+<template>
+  <div>{{text}}</div>
+</template>
+```
+
+:-1: Examples of **incorrect** code`:
+
+```html
+<template>
+  <div>{{ text }}</div>
+</template>
+```

docs/rules/no-multi-spaces.md

@@ -0,0 +1,31 @@
+# This rule warns about the usage of extra whitespaces between attributes (no-multi-spaces)
+
+The `--fix` option on the command line can automatically fix some of the problems reported by this rule.
+
+This rule aims to remove multiple spaces in a row between attributes witch are not used for indentation.
+
+## Rule Details
+
+Examples of **incorrect** code for this rule:
+
+```html
+<template>
+  <div class="foo"      :style="foo"
+    :foo="bar"         >
+  </div>
+</template>
+```
+
+Examples of **correct** code for this rule:
+
+```html
+<template>
+  <div class="foo"
+    :style="foo">
+  </div>
+</template>
+```
+
+### Options
+
+Nothing

docs/rules/no-reserved-keys.md

@@ -0,0 +1,58 @@
+# Prevent overwrite reserved keys (no-reserved-keys)
+
+This rule prevents to use reserved names from to avoid conflicts and unexpected behavior.
+
+## Rule Details
+
+:-1: Examples of **incorrect** code for this rule:
+
+```js
+export default {
+  props: {
+    $el: String
+  },
+  computed: {
+    $on: {
+      get () {
+      }
+    }
+  },
+  data: {
+    _foo: null
+  },
+  methods: {
+    $nextTick () {
+    }
+  }
+}
+```
+
+## :wrench: Options
+
+This rule has an object option:
+
+`"reserved"`: [] (default) array of dissalowed names inside `groups`.
+
+`"groups"`: [] (default) array of additional groups to search for duplicates.
+
+### Example:
+
+```
+vue/no-reserved-keys: [2, {
+  reserved: ['foo', 'foo2'],
+  groups: ['asyncComputed']
+}]
+```
+
+:-1: Examples of **incorrect** code for this configuration
+
+```js
+export default {
+  asyncComputed: {
+    foo2 () {}
+  },
+  computed: {
+    foo () {}
+  }
+}
+```

docs/rules/require-default-prop.md

@@ -0,0 +1,45 @@
+# Require default value for props (require-default-prop)
+
+This rule requires default value to be set for each props that are not marked as `required`.
+
+## Rule Details
+
+Examples of **incorrect** code for this rule:
+
+```js
+export default {
+  props: {
+    a: Number,
+    b: [Number, String],
+    c: {
+      type: Number
+    },
+    d: {
+      type: Number,
+      required: false
+    }
+  }
+}
+```
+
+Examples of **correct** code for this rule:
+
+```js
+export default {
+  props: {
+    a: {
+      type: Number,
+      required: true
+    },
+    b: {
+      type: Number,
+      default: 0
+    },
+    c: {
+      type: Number,
+      default: 0,
+      required: false
+    }
+  }
+}
+```

docs/rules/require-render-return.md

@@ -1,6 +1,6 @@
 # Enforces render function to always return value (require-render-return)
 
-This rule aims to enforce render function to allways return value
+This rule aims to enforce render function to always return value
 
 ## :book: Rule Details