-
Notifications
You must be signed in to change notification settings - Fork 532
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add format and output options to CLI #152
Conversation
Wanted to debug travis mainly here with a build.
Test failing could be to do with the issue in #151, i doubt it very much but will wait to fix this once that has been merged in. |
Looking at the Travis output, it's installing sass-lint globally at 1.2.0, is the current branch getting merged into that? If not that will be the reasons for the test failing as the new functionality doesn't exist.. |
Now also closes #181 Added another test case and fixed the failing tests. |
Looks good. |
Add format and output options to CLI
Add format and output options to CLI
Added 2 new flags to the CLI
-f --format [format]
-o --output [output]
This makes the CLI feature complete compared to the current
sass-lint
build. Format allows you to specify an output format from any of the eslint formatters. Output allows you to specify and output directory & file to write linting results to.Also added tests for each option.
There's a slight overlap in some of the files added here as in #151 which should probably be merged first due to it fixing bugs.
closes #127
DCO 1.1 Signed-off-by: Dan Purdy danjpurdy@gmail.com