Skip to content
This repository was archived by the owner on Jun 20, 2023. It is now read-only.

switch to js based sass linter #2488

Merged
merged 6 commits into from
Jul 7, 2016
Merged

switch to js based sass linter #2488

merged 6 commits into from
Jul 7, 2016

Conversation

xi
Copy link
Collaborator

@xi xi commented Jul 6, 2016

Following #2486, this replaces the ruby based scss-lint by node based sass-lint.

sass-lint does not yet support inline comments to disable specific rules (there is a pull request though: sasstools/sass-lint#677). I switched the affected rules to warning mode, meaning they will provide warning messages, but the process will still exit with exitcode 0.

ignore: '**/third_party/**/*.scss'
options:
formatter: stylish
merge-default-rules: true
Copy link

@der-john der-john Jul 7, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I got merge-default-rules: false when I pasted our scss lint config on https://sasstools.github.io/make-sass-lint-config/ .

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, I adapted the output manually.

@der-john
Copy link

der-john commented Jul 7, 2016

looks good & works.

@der-john der-john merged commit 39ab337 into master Jul 7, 2016
@joka joka deleted the 2016-07-tb-js-sass-lint branch April 12, 2018 11:27
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants