Skip to content

Commit

Permalink
Merge pull request #32 from jGleitz/sass-lint
Browse files Browse the repository at this point in the history
Add sass linting
  • Loading branch information
valentinz committed Nov 15, 2015
2 parents 247534d + 419aa7c commit 07d2545
Show file tree
Hide file tree
Showing 4 changed files with 229 additions and 4 deletions.
14 changes: 14 additions & 0 deletions .editorconfig
Original file line number Diff line number Diff line change
@@ -0,0 +1,14 @@
root = true

[**/*]
indent_style = tab
end_of_line = lf
charset = utf-8

[package.json]
indent_style = space
indent_size = 2

[**/*.yml]
indent_style = space
indent_size = 4
210 changes: 210 additions & 0 deletions css/.sass-lint.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,210 @@
options:
formatter: stylish


files:
include: 'css/**/*.scss'


rules:
border-zero:
- 1
- convention: none

brace-style:
- 1
- style: 1tbs
allow-single-line: false

clean-import-paths:
- 1
- leading-underscore: false
filename-extension: false

empty-args:
- 1
- include: false

empty-line-between-blocks:
- 1
- include: true
allow-single-line-rulesets: false

extends-before-declarations:
- 1

extends-before-mixins:
- 1

final-newline:
- 1
- include: true

force-attribute-nesting:
- 0

force-element-nesting:
- 0

foce-pseudo-nesting:
- 0

function-name-format:
- 1
- allow-leading-underscore: true
convention: hyphenatedlowercase

hex-length:
- 1
- style: short

hex-notation:
- 1
- style: uppercase

indentation:
- 1
- size: 1

leading-zero:
- 1
- include: false

mixin-name-format:
- 1
- allow-leading-underscore: true
convention: hyphenatedlowercase

mixins-before-declarations:
- 1

nesting-depth:
- 2
- max-depth: 4

no-color-keywords:
- 0

no-color-literals:
- 2
- allow-rgba: true

no-css-comments:
- 0

no-debug:
- 0

no-duplicate-properties:
- 2

no-empty-rulesets:
- 1

no-extends:
- 0

no-ids:
- 0

no-important:
- 2

no-invalid-hex:
- 2

no-mergable-selectors:
- 0

no-misspelled-properties:
- 2

no-qualifying-elements:
- 0

no-trailing-zero:
- 1

no-transition-all:
- 1

no-url-protocols:
- 1

no-vendor-prefixes:
- 2

no-warn:
- 0

one-declaration-per-line:
- 1

placeholder-in-extends:
- 0

placeholder-name-format:
- 1
- allow-leading-underscore: true
convention: hyphenatedlowercase

property-sort-order:
- 1
- order: concentric

quotes:
- 1
- style: single

shorthand-values:
- 0

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
- include: false

space-between-parens:
- 1
- include: false

trailing-semicolon:
- 1
- include: true

url-quotes:
- 1

variable-for-property:
- 0

variable-name-format:
- 1
- allow-leading-underscore: true
convention: hyphenatedlowercase

zero-unit:
- 1
- include: false
6 changes: 3 additions & 3 deletions css/style.scss
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
body {
margin-top: 10px;
margin-top: 10px;
}

.angular-google-map-container {
height: 520px;
}
height: 520px;
}
3 changes: 2 additions & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -47,6 +47,7 @@
"portfinder": "^0.4.0",
"protractor": "git+https://github.com/jGleitz/protractor.git#browserstack",
"q": "^1.4.1",
"sass-lint": "^1.3.2",
"serve-static": "^1.10.0",
"watch": "^0.16.0"
},
Expand All @@ -56,8 +57,8 @@
"test": "protractor test/behaviour/protractor.conf",
"install-testdependencies": "webdriver-manager update --standalone --chrome --browserstacklocal",
"browsertest": "protractor test/behaviour/protractor.browserstack.conf",
"lint": "eslint js/**/*.js test/**/*.js && sass-lint -c css/.sass-lint.yml -v",
"chrometest": "protractor test/behaviour/protractor.conf --browser chrome",
"lint": "eslint js/**/*.js test/**/*.js",
"start": "node tasks/watch"
}
}

0 comments on commit 07d2545

Please sign in to comment.