Skip to content

Conversation

SimenB
Copy link
Contributor

@SimenB SimenB commented Jul 21, 2015

Fixes #5 #30

Even if CSSlint fixes it so they don't explode, there's no reason we can't short-circuit

@lazd
Copy link
Owner

lazd commented Aug 6, 2015

I commented on the PR at CSSLint/csslint, so hopefully one of the maintainers will pick it up and patch this in the project itself. @nschonni? @XhmikosR?

This is the best we can do for now, will merge and release.

lazd added a commit that referenced this pull request Aug 6, 2015
Don't send empty files to csslint
@lazd lazd merged commit b365adf into lazd:master Aug 6, 2015
@SimenB SimenB deleted the empty-files branch August 6, 2015 15:08
@SimenB
Copy link
Contributor Author

SimenB commented Aug 6, 2015

CSSLint hasn't had a release in 2 years, so I think this is the way to go for now 😄

@lazd
Copy link
Owner

lazd commented Aug 6, 2015

@SimenB very true, though there has been a little activity this year! Maybe a release is coming...

@XhmikosR
Copy link

XhmikosR commented Aug 6, 2015

Hi, there.

I'm not a CSSLint maintainer so I can't really help here.
On Aug 6, 2015 6:08 PM, "Larry Davis" notifications@github.com wrote:

I commented on the PR at CSSLint/csslint, so hopefully one of the
maintainers will pick it up and patch this in the project itself.
@nschonni https://github.com/nschonni? @XhmikosR
https://github.com/XhmikosR?

This is the best we can do for now, will merge and release.


Reply to this email directly or view it on GitHub
#32 (comment).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Lint fails if file is empty

3 participants