Skip to content

Commit

Permalink
Upgrade from scss-lint to sass-lint (Hacker0x01#697)
Browse files Browse the repository at this point in the history
* Upgrade from scss-lint to sass-lint

sass-lint is pure JS, so this removes the Ruby dependency.

* Also upgrade from grunt-contrib-sass to grunt-sass

grunt-sass uses node-sass, whereas grunt-contrib-sass
still depends on Ruby.
  • Loading branch information
aij authored and martijnrusschen committed Jan 19, 2017
1 parent 33ad8e4 commit faa5f4c
Show file tree
Hide file tree
Showing 5 changed files with 163 additions and 227 deletions.
154 changes: 154 additions & 0 deletions .sass-lint.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,154 @@
# sass-lint config generated by make-sass-lint-config v0.1.2
#
# The following scss-lint Linters are not yet supported by sass-lint:
# ChainedClasses, DisableLinterReason, ElsePlacement, PropertyCount
# PseudoElement, SelectorDepth, SpaceAroundOperator, TrailingWhitespace
# UnnecessaryParentReference, Compass::*
#
# The following settings/values are unsupported by sass-lint:
# Linter HexLength, option "enaled"
# Linter Indentation, option "allow_non_nested_indentation"
# Linter Indentation, option "character"
# Linter SpaceBeforeBrace, option "allow_single_line_padding"

files:
include: '**/*.scss'
options:
formatter: stylish
merge-default-rules: false
rules:
bem-depth:
- 0
- max-depth: 1
border-zero: 0
brace-style: 1
class-name-format:
- 1
- convention: hyphenatedbem
clean-import-paths:
- 1
- filename-extension: false
leading-underscore: false
empty-line-between-blocks:
- 1
- ignore-single-line-rulesets: true
extends-before-declarations: 1
extends-before-mixins: 1
final-newline:
- 1
- include: true
force-attribute-nesting: 1
force-element-nesting: 1
force-pseudo-nesting: 1
function-name-format:
- 0
- convention: BEM
hex-length:
- 1
- style: short
hex-notation:
- 1
- style: lowercase
id-name-format:
- 1
- convention: hyphenatedbem
indentation:
- 1
- size: 2
leading-zero:
- 1
- include: false
mixin-name-format:
- 0
- convention: BEM
mixins-before-declarations: 1
nesting-depth:
- 1
- max-depth: 4
no-color-keywords: 1
no-color-literals: 0
no-css-comments: 0
no-debug: 1
no-duplicate-properties: 1
no-empty-rulesets: 1
no-extends: 0
no-ids: 1
no-important: 1
no-invalid-hex: 1
no-mergeable-selectors: 1
no-misspelled-properties: 1
no-qualifying-elements:
- 1
- allow-element-with-attribute: false
allow-element-with-class: false
allow-element-with-id: false
no-trailing-zero: 1
no-transition-all: 0
no-url-protocols: 1
no-vendor-prefixes: 0
placeholder-in-extend: 1
placeholder-name-format:
- 1
- convention: hyphenatedbem
property-sort-order: 0
property-units:
- 1
- global:
- ch
- em
- ex
- rem
- cm
- in
- mm
- pc
- pt
- px
- q
- vh
- vw
- vmin
- vmax
- deg
- grad
- rad
- turn
- ms
- s
- Hz
- kHz
- dpi
- dpcm
- dppx
- '%'
quotes:
- 1
- style: double
shorthand-values: 1
single-line-per-selector: 1
space-after-bang:
- 1
- include: false
space-after-colon:
- 1
- include: true
space-after-comma:
- 1
- include: true
space-before-bang:
- 1
- include: true
space-before-brace:
- 1
- include: true
space-before-colon: 1
space-between-parens:
- 1
- include: false
trailing-semicolon: 1
url-quotes: 1
variable-for-property: 0
variable-name-format:
- 0
- convention: BEM
zero-unit: 1
216 changes: 0 additions & 216 deletions .scss-lint.yml

This file was deleted.

2 changes: 0 additions & 2 deletions .travis.yml
Original file line number Diff line number Diff line change
Expand Up @@ -11,9 +11,7 @@ addons:
packages:
- g++-4.8
before_install:
- rvm install 2.3.1
- npm install -g grunt-cli
- gem update --system && gem install scss_lint -v 0.50.2
- export DISPLAY=:99.0
- sh -e /etc/init.d/xvfb start
script:
Expand Down
Loading

0 comments on commit faa5f4c

Please sign in to comment.