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.

Sign up for GitHub

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

Jump to bottom

Improve lint #312

Closed

vecmezoni wants to merge 1 commit into csscomb:dev from vecmezoni:lint

Contributor

vecmezoni commented Sep 13, 2014

This PR is still in progress, just wanted to review this proof of concept.

Partially fixes #84 #135 #245


          improve lint

1aa753b

Member

tonyganch commented Sep 14, 2014

There are two more issues you can find helpful: #261 and csscomb/core#4

New methods should not be static, so the changes should be done in csscomb/core repo.
Some changes should be done in Plugins API: plugins should have a separate lint method and lintMessage field for user-friendly error messages.
You can find an example of what lint... methods should return in this comment.
Method that formats error messages (displays a few lines before/after and points where error is) should live in this repo.

I can help you with all of these if you want.

tonyganch closed this

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

Labels

None yet