From: Geoff Kimball Date: Tue, 16 Feb 2016 01:05:19 +0000 (-0800) Subject: Replace scss-lint with sass-lint X-Git-Tag: v2.0.0-rc.1~7 X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=b02a51a98dc77e0cb19b29e254c07f862d181044;p=thirdparty%2Ffoundation%2Ffoundation-emails.git Replace scss-lint with sass-lint --- diff --git a/.sass-lint.yml b/.sass-lint.yml new file mode 100644 index 00000000..53b72fa1 --- /dev/null +++ b/.sass-lint.yml @@ -0,0 +1,116 @@ +files: + include: 'scss/**/*.scss' +options: + formatter: stylish + merge-default-rules: false +rules: + border-zero: + - 1 + - convention: zero + brace-style: + - 1 + - allow-single-line: false + class-name-format: + - 1 + - convention: '([a-z0-9]+-?)+' + 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: + - 1 + - allow-leading-underscore: true + convention: hyphenatedlowercase + hex-length: + - 1 + - style: short + hex-notation: + - 1 + - style: lowercase + id-name-format: + - 1 + - convention: '([a-z0-9]+-?)+' + indentation: + - 1 + - size: 2 + leading-zero: + - 1 + - include: true + mixin-name-format: + - 1 + - allow-leading-underscore: true + convention: hyphenatedlowercase + mixins-before-declarations: 1 + nesting-depth: + - 1 + - max-depth: 3 + no-color-keyword: 1 + no-color-literals: 1 + no-css-comments: 0 + no-debug: 1 + no-duplicate-properties: 1 + no-empty-rulesets: 1 + no-ids: 1 + no-important: 0 + no-invalid-hex: 1 + no-mergeable-selectors: 1 + no-misspelled-properties: + - 1 + - extra-properties: [] + no-qualifying-elements: + - 0 + - allow-element-with-attribute: false + allow-element-with-class: false + allow-element-with-id: false + no-trailing-zero: 1 + no-url-protocols: 1 + placeholder-in-extend: 1 + placeholder-name-format: + - 1 + - convention: '([a-z0-9]+-?)+' + property-sort-order: + - 0 + - ignore-custom-properties: false + quotes: + - 1 + - style: single + shorthand-values: 1 + single-line-per-selector: 0 + space-after-bang: + - 1 + - include: false + space-after-colon: + - 1 + - include: true + space-after-comma: 1 + 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 + - properties: [] + variable-name-format: + - 1 + - allow-leading-underscore: true + convention: hyphenatedlowercase + zero-unit: 1 diff --git a/config/scss-lint.yml b/config/scss-lint.yml deleted file mode 100644 index 2e4056b6..00000000 --- a/config/scss-lint.yml +++ /dev/null @@ -1,189 +0,0 @@ -# Default application configuration that all configurations inherit from. - -scss_files: "scss/**/*.scss" - -linters: - BangFormat: - enabled: true - space_before_bang: true - space_after_bang: false - - BorderZero: - enabled: true - convention: zero # or `none` - - ColorKeyword: - enabled: true - - ColorVariable: - enabled: true - - Comment: - enabled: false - - DebugStatement: - enabled: true - - DeclarationOrder: - enabled: true - - DuplicateProperty: - enabled: true - - ElsePlacement: - enabled: true - style: new_line # or 'same_line' - - EmptyLineBetweenBlocks: - enabled: true - ignore_single_line_blocks: true - - EmptyRule: - enabled: true - - FinalNewline: - enabled: true - present: true - - HexLength: - enabled: true - style: short # or 'long' - - HexNotation: - enabled: true - style: lowercase # or 'uppercase' - - HexValidation: - enabled: true - - IdSelector: - enabled: true - - ImportantRule: - enabled: false - - ImportPath: - enabled: true - leading_underscore: false - filename_extension: false - - Indentation: - enabled: true - allow_non_nested_indentation: false - character: space # or 'tab' - width: 2 - - LeadingZero: - enabled: true - style: include_zero # or 'include_zero' - - MergeableSelector: - enabled: true - force_nesting: true - - NameFormat: - enabled: true - allow_leading_underscore: true - convention: hyphenated_lowercase # or 'BEM', or a regex pattern - - NestingDepth: - enabled: true - max_depth: 3 - - PlaceholderInExtend: - enabled: true - - PropertyCount: - enabled: false - include_nested: false - max_properties: 10 - - PropertySortOrder: - enabled: false - ignore_unspecified: false - separate_groups: false - - PropertySpelling: - enabled: true - extra_properties: [] - - QualifyingElement: - enabled: false - allow_element_with_attribute: false - allow_element_with_class: false - allow_element_with_id: false - - SelectorDepth: - enabled: true - max_depth: 3 - - SelectorFormat: - enabled: true - convention: '([a-z0-9]+-?)+' - - Shorthand: - enabled: true - - SingleLinePerProperty: - enabled: true - allow_single_line_rule_sets: false - - SingleLinePerSelector: - enabled: false - - SpaceAfterComma: - enabled: true - - SpaceAfterPropertyColon: - enabled: true - style: one_space # or 'no_space', or 'at_least_one_space', or 'aligned' - - SpaceAfterPropertyName: - enabled: true - - SpaceBeforeBrace: - enabled: true - style: space # or 'new_line' - allow_single_line_padding: true - - SpaceBetweenParens: - enabled: true - spaces: 0 - - StringQuotes: - enabled: true - style: single_quotes # or double_quotes - - TrailingSemicolon: - enabled: true - - TrailingZero: - enabled: false - - UnnecessaryMantissa: - enabled: true - - UnnecessaryParentReference: - enabled: true - - UrlFormat: - enabled: true - - UrlQuotes: - enabled: true - - VariableForProperty: - enabled: false - properties: [] - - VendorPrefixes: - enabled: true - identifier_list: base - include: [] - exclude: [] - - ZeroUnit: - enabled: true - - Compass::*: - enabled: false \ No newline at end of file diff --git a/gulpfile.js b/gulpfile.js index a1b4e860..cea3712d 100644 --- a/gulpfile.js +++ b/gulpfile.js @@ -65,9 +65,9 @@ gulp.task('settings', function() { gulp.task('lint', function() { return gulp.src('scss/**/*.scss') - .pipe($.scssLint({ - 'config': 'config/scss-lint.yml' - })); + .pipe($.sassLint()) + .pipe($.sassLint.format()) + .pipe($.sassLint.failOnError()); }); gulp.task('server', ['build'], function() { diff --git a/package.json b/package.json index 56e55673..c8d813a2 100644 --- a/package.json +++ b/package.json @@ -22,7 +22,7 @@ "gulp-load-plugins": "^1.0.0-rc.1", "gulp-ruby-sass": "^1.0.5", "gulp-sass": "^2.1.0", - "gulp-scss-lint": "^0.2.4", + "gulp-sass-lint": "^1.1.1", "gulp-sourcemaps": "^1.6.0", "gulp-webserver": "^0.9.1", "handlebars": "^3.0.3",