summaryrefslogtreecommitdiff
path: root/Gemfile
diff options
context:
space:
mode:
authorJacob Schatz <jschatz1@gmail.com>2016-03-15 16:52:53 +0000
committerJacob Schatz <jschatz1@gmail.com>2016-03-15 16:52:53 +0000
commit37707ac59e6c4e6db2b758338e13cb089dacd8c7 (patch)
treeb78a0d665fafde7b9d1ad6a3cc7795a3bc2b2e17 /Gemfile
parent0602091f0cdebbc3183732dee78c38f89b4b7d01 (diff)
parent8d8b457cebdfd0790157cd54fd1f24e46fbf0785 (diff)
downloadgitlab-ce-37707ac59e6c4e6db2b758338e13cb089dacd8c7.tar.gz
Merge branch 'css-style-guide' into 'master'
CSS style guide Working towards what was discussed in #13552, this adds the [SCSS Linter gem](https://github.com/brigade/scss-lint) for style guide conformity in CI. TODO: - [x] Agree on and write SCSS Style Guide Documentation. - [x] Document the `scss-lint` config file. - [x] Figure out how best to run this in CI, right now it's taking longer than I would think it should. - [x] Use CSSComb for auto-correction (Maybe just include a CSSComb config file and have developers run the node package manually if they're interested in using it?). My logic for not using CSSComb in the first place is that, AFAIK, we don't currently require Node/NPM, even in the dev environment. Maybe I'm wrong about that? `scss-lint` is a Ruby implementation of a similar concept, but it doesn't include autocorrect. Is there a way we can run get GitLab CI to run `scss-lint` without having to put together the full GitLab application? Seeing as it's just static analysis, it seems like a waste of time/resources. /cc @jschatz1 @rspeicher See merge request !3069
Diffstat (limited to 'Gemfile')
-rw-r--r--Gemfile1
1 files changed, 1 insertions, 0 deletions
diff --git a/Gemfile b/Gemfile
index 2f14b9965b2..a0e8e796627 100644
--- a/Gemfile
+++ b/Gemfile
@@ -283,6 +283,7 @@ group :development, :test do
gem 'spring-commands-teaspoon', '~> 0.0.2'
gem 'rubocop', '~> 0.35.0', require: false
+ gem 'scss_lint', '~> 0.47.0', require: false
gem 'coveralls', '~> 0.8.2', require: false
gem 'simplecov', '~> 0.10.0', require: false
gem 'flog', require: false