Skip to content
This repository has been archived by the owner on Aug 7, 2023. It is now read-only.

Use JSON scss-lint formatter, add additionalArguments option #42

Merged
merged 1 commit into from
Jun 26, 2015

Conversation

caseywebdev
Copy link
Contributor

The XML format is not longer supported in the latest scss-lint (see the scss-lint CHANGELOG). This change uses the preferred JSON formatter. It also swaps the excludeLinters option for an additionalArguments option, which gives users the freedom to add whatever command line arguments they want, not just excludeLinters (includeLinters, for example).

@dsandstrom
Copy link

Thanks, this works well.

@caseywebdev
Copy link
Contributor Author

Sure thing. I've been using this for a few weeks now without issue. The latest official version simply does not work at all with the latest scss_lint because of the XML format removal.

@FalkoJoseph
Copy link

That's the fix 👍 works great for me!

@FalkoJoseph FalkoJoseph mentioned this pull request Jun 18, 2015
@brunowego
Copy link

Works great 👍

@kristerkari
Copy link

This works 👍

iam4x pushed a commit that referenced this pull request Jun 26, 2015
Use JSON scss-lint formatter, add additionalArguments option
@iam4x iam4x merged commit e4b337b into AtomLinter:master Jun 26, 2015
@caseywebdev caseywebdev deleted the json branch July 24, 2015 16:45
@steelbrain
Copy link
Contributor

@iam4x while this works, It would've been ideal to have it completely upgraded to the latest API as we had discussed about removing this API in linter-dev and the change was mostly supported, but was still rejected because it would break a lot of linters :)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants