Commit efe49ae6 authored by Robert Speicher's avatar Robert Speicher

Merge branch 'ce-to-ee-2018-10-10' into 'master'

CE upstream - 2018-10-10 14:21 UTC

See merge request gitlab-org/gitlab-ee!7865
parents 4eaac005 0d2f5da1
...@@ -156,7 +156,7 @@ the feature you contribute through all of these steps. ...@@ -156,7 +156,7 @@ the feature you contribute through all of these steps.
1. Performance/scalability implications have been considered, addressed, and tested 1. Performance/scalability implications have been considered, addressed, and tested
1. [Documented][doc-guidelines] in the `/doc` directory 1. [Documented][doc-guidelines] in the `/doc` directory
1. [Changelog entry added][changelog], if necessary 1. [Changelog entry added][changelog], if necessary
1. Reviewed and any concerns are addressed 1. Reviewed by UX/FE/BE and any concerns are addressed
1. Merged by a project maintainer 1. Merged by a project maintainer
1. Added to the release blog article, if relevant 1. Added to the release blog article, if relevant
1. Added to [the website](https://gitlab.com/gitlab-com/www-gitlab-com/), if relevant 1. Added to [the website](https://gitlab.com/gitlab-com/www-gitlab-com/), if relevant
......
# Style guides and linting # Style guides and linting
See the relevant style guides for our guidelines and for information on linting: See the relevant style guides for our guidelines and for information on linting:
## JavaScript ## JavaScript
We defer to [Airbnb][airbnb-js-style-guide] on most style-related We defer to [Airbnb][airbnb-js-style-guide] on most style-related
conventions and enforce them with eslint. conventions and enforce them with eslint.
See [our current .eslintrc][eslintrc] for specific rules and patterns. See [our current .eslintrc](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/.eslintrc.yml) for specific rules and patterns.
### Common ### Common
...@@ -299,10 +301,12 @@ Do not use them anymore and feel free to remove them when refactoring legacy cod ...@@ -299,10 +301,12 @@ Do not use them anymore and feel free to remove them when refactoring legacy cod
### Vue.js ### Vue.js
#### `eslint-vue-plugin` #### `eslint-vue-plugin`
We default to [eslint-vue-plugin][eslint-plugin-vue], with the `plugin:vue/recommended`. We default to [eslint-vue-plugin][eslint-plugin-vue], with the `plugin:vue/recommended`.
Please check this [rules][eslint-plugin-vue-rules] for more documentation. Please check this [rules][eslint-plugin-vue-rules] for more documentation.
#### Basic Rules #### Basic Rules
1. The service has it's own file 1. The service has it's own file
1. The store has it's own file 1. The store has it's own file
1. Use a function in the bundle file to instantiate the Vue component: 1. Use a function in the bundle file to instantiate the Vue component:
...@@ -387,6 +391,7 @@ Please check this [rules][eslint-plugin-vue-rules] for more documentation. ...@@ -387,6 +391,7 @@ Please check this [rules][eslint-plugin-vue-rules] for more documentation.
[#34371]: https://gitlab.com/gitlab-org/gitlab-ce/issues/34371 [#34371]: https://gitlab.com/gitlab-org/gitlab-ce/issues/34371
#### Alignment #### Alignment
1. Follow these alignment styles for the template method: 1. Follow these alignment styles for the template method:
1. With more than one attribute, all attributes should be on a new line: 1. With more than one attribute, all attributes should be on a new line:
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment