Skip to content
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

Feature/write output to file #107

Merged
merged 5 commits into from
Sep 5, 2015
Merged

Conversation

Snugug
Copy link
Member

@Snugug Snugug commented Sep 5, 2015

Resolves #48

DCO 1.1 Signed-off-by: Sam Richard <snugug@gmail.com>

@Snugug Snugug added this to the 1.2.0 milestone Sep 5, 2015
@DanPurdy
Copy link
Member

DanPurdy commented Sep 5, 2015

Are there any plans to add defining an output directory?

@Snugug
Copy link
Member Author

Snugug commented Sep 5, 2015

I had only considered outputting results to a file, not a way of outputting multiple files to a directory

@DanPurdy
Copy link
Member

DanPurdy commented Sep 5, 2015

Makes sense, this is great though. Eslint formatters will be a massive plus to people.

Great stuff, @bgriffith you should give this a once over too

@Snugug
Copy link
Member Author

Snugug commented Sep 5, 2015

Haha, the use of ESLint formatters was mostly because I'm too lazy to write my own (and for the life of me I can't figure out why they haven't been blown out in to their own packages yet)

@DanPurdy
Copy link
Member

DanPurdy commented Sep 5, 2015

Ha yeah, if they aint broken why fix and all that...

@bgriffith
Copy link
Member

Looking good.

bgriffith added a commit that referenced this pull request Sep 5, 2015
@bgriffith bgriffith merged commit ca15283 into master Sep 5, 2015
@Snugug Snugug deleted the feature/write-output-to-file branch September 6, 2015 13:04
donabrams pushed a commit to donabrams/sass-lint that referenced this pull request Nov 19, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants