Add the following JSON to your .arclint:
"scss-lint": { "type": "scss-lint" }
Then, add a ruby file with errors, for instance:
.message { margin: 0; margin: 10px; }
Run arc lint. It should come up with: "Property margin already defined on line 2"
Details
Details
- Reviewers
remon epriestley chad - Group Reviewers
Blessed Reviewers
Diff Detail
Diff Detail
- Repository
- rARC Arcanist
- Branch
- add-scss-linter
- Lint
Lint Passed - Unit
Tests Passed - Build Status
Buildable 2882 Build 2886: [Placeholder Plan] Wait for 30 Seconds
Event Timeline
| src/lint/linter/__tests__/ArcanistLinterTestCase.php | ||
|---|---|---|
| 7 | We don't want this... the expectation is that scss-lint $SOME_PATH should work regardless of the file extension. | |
Comment Actions
Anyone have an update on this? I'd love to use it in our project.
| src/lint/linter/ArcanistSCSSLintLinter.php | ||
|---|---|---|
| 13 | This now lives at: https://github.com/brigade/scss-lint | |
Comment Actions
Sorry for not responding! Is there something you need from me in the diff? I'll be happy to help!
Comment Actions
Any plan on taking this issue over? Should I try? This would be pretty usefull for us :)
Comment Actions
You could use a Script and Regex Linter with config like this:
"stylelint": { "type": "script-and-regex", "script-and-regex.script": "sh -c 'node ./node_modules/stylelint/bin/stylelint.js \"$0\" || true'", "script-and-regex.regex": "/^(\\h+)(?P<line>\\d+):(?P<char>\\d+)(\\h+)(?P<error>✖)(\\h+)(?P<message>.*)(\\h{1,})(?P<code>.*)$/m", "include": "/local\\/assets\\/styles\\/.*\\.scss/i" }