summaryrefslogtreecommitdiff
path: root/.scss-lint.yml
Commit message (Collapse)AuthorAgeFilesLines
* Enable ColorVariable in scss-lintSam Rose2016-12-011-1/+1
|
* Enable PseudoElement in scss-lintenable-scss-lint-pseudo-elementClement Ho2016-10-261-1/+1
|
* Merge branch 'enable-scss-lint-single-line-per-selector' into 'master' Annabel Dunstone Gray2016-10-251-1/+1
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Enable SingleLinePerSelector in scss-lint ## What does this MR do? Enables `SingleLinePerSelector` in `.scss-lint.yml` ## Are there points in the code the reviewer needs to double check? This MR touches a lot of files, just need another pair of eyes to look through and make sure I didn't break anything. ## Why was this MR needed? Improve SCSS maintainability and consistency ## Screenshots (if relevant) None ## Does this MR meet the acceptance criteria? - Tests - [x] Added for this feature/bug - [x] All builds are passing - [x] Conform by the [merge request performance guides](http://docs.gitlab.com/ce/development/merge_request_performance_guidelines.html) - [x] Conform by the [style guides](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/CONTRIBUTING.md#style-guides) - [x] Branch has no merge conflicts with `master` (if it does - rebase it please) - [x] [Squashed related commits together](https://git-scm.com/book/en/Git-Tools-Rewriting-History#Squashing-Commits) ## What are the relevant issue numbers? #23485 See merge request !7081
| * Enable SingleLinePerSelector in scss-lintenable-scss-lint-single-line-per-selectorClement Ho2016-10-241-1/+1
| |
* | Merge branch 'enable-scss-lint-space-after-variable-colon' into 'master' Annabel Dunstone Gray2016-10-241-1/+1
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Enable SpaceAfterVariableColon in scss-lint ## What does this MR do? Enables `SpaceAfterVariableColon` in `.scss-lint.yml` ## Are there points in the code the reviewer needs to double check? None ## Why was this MR needed? Improve SCSS maintainability and consistency ## Screenshots (if relevant) None ## Does this MR meet the acceptance criteria? - Tests - [x] All builds are passing - [x] Conform by the [merge request performance guides](http://docs.gitlab.com/ce/development/merge_request_performance_guidelines.html) - [x] Conform by the [style guides](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/CONTRIBUTING.md#style-guides) - [x] Branch has no merge conflicts with `master` (if it does - rebase it please) - [x] [Squashed related commits together](https://git-scm.com/book/en/Git-Tools-Rewriting-History#Squashing-Commits) ## What are the relevant issue numbers? #23485 See merge request !7080
| * | Enable SpaceAfterVariableColon in scss-lintenable-scss-lint-space-after-variable-colonClement Ho2016-10-241-1/+1
| |/
* | Merge branch 'enable-scss-lint-space-around-operator' into 'master' Annabel Dunstone Gray2016-10-241-1/+1
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Enable SpaceAroundOperator in scss-lint ## What does this MR do? Enables `SpaceAroundOperator` in .`scss-lint.yml` ## Are there points in the code the reviewer needs to double check? None ## Why was this MR needed? Improve SCSS maintainability and consistency ## Screenshots (if relevant) None ## Does this MR meet the acceptance criteria? - Tests - [x] All builds are passing - [x] Conform by the [merge request performance guides](http://docs.gitlab.com/ce/development/merge_request_performance_guidelines.html) - [x] Conform by the [style guides](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/CONTRIBUTING.md#style-guides) - [x] Branch has no merge conflicts with `master` (if it does - rebase it please) - [x] [Squashed related commits together](https://git-scm.com/book/en/Git-Tools-Rewriting-History#Squashing-Commits) ## What are the relevant issue numbers? #23485 See merge request !7078
| * | Enable SpaceAroundOperator in scss-lintClement Ho2016-10-241-1/+1
| |/
* | Enable trailingWhitespace in scss-lintClement Ho2016-10-241-1/+1
|/
* Enforce TrailingSemicolon and EmptyLineBetweenBlocks in scss-lint23319-enable-trailingsemicolon-on-scss-lintClement Ho2016-10-141-2/+2
|
* Remove SCSS rules for short hex chars.Jacob Schatz2016-10-031-1/+1
|
* Merge branch 'scss-url-linters' into 'master' Jacob Schatz2016-04-281-2/+2
|\ | | | | | | | | | | | | | | | | | | | | | | | | Enable the UrlFormat and UrlQuotes SCSS Linters. There were no lints to fix for this one, so it's just preventative. UrlFormat: URLs should be valid and not contain protocols or domain names. UrlQuotes: URLs should always be enclosed within quotes. cc: @jschatz1 See merge request !3770
| * Enable the UrlFormat and UrlQuotes SCSS Linters.Connor Shea2016-04-161-2/+2
| |
* | Enable the EmptyRule SCSS Linter.Connor Shea2016-04-161-1/+1
|/ | | | | | Prevents empty SCSS rules from being included in the SCSS. Also fixes the handful of lints caught by the linter.
* Add comments to the SCSS Lint config file [ci skip]connorshea2016-04-041-0/+105
| | | | Also add some previously missing linters.
* Enable SpaceBeforeBrace rule for SCSS Lintconnorshea2016-03-301-1/+2
| | | | | Also resolve all instances of spaces not being included before opening braces.
* Disabled SelectorFormat rule for SCSS Lint in order to get the Rake task ↵connorshea2016-03-181-1/+1
| | | | | | | | passing sooner. Discussed in #14299. [ci skip]
* Add SCSS Lint, CSSComb config file, run SCSS Lint in GitLab CI, add ↵connorshea2016-03-141-0/+158
documentation for SCSS Style Guide. See !3069 for more information.