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

css linting #722

Open
pascalchevrel opened this issue Apr 12, 2016 · 3 comments
Open

css linting #722

pascalchevrel opened this issue Apr 12, 2016 · 3 comments
Labels

Comments

@pascalchevrel
Copy link
Member

Now that we lint both php and js, let's discuss if there is a way to also lint our css files without adding too complex dependencies to the project.

@flodolo
Copy link
Collaborator

flodolo commented Apr 12, 2016

Adding csslint would be really straightforward (it's a node package like eslint), the problem with CSS linters is that they tend to be extremely semantic, not just care about syntax, so it would be tricky to create a set of rules without redesigning half of the templates.

Example: csslint run against /web/styles without any specific config: result.txt

@pascalchevrel
Copy link
Member Author

I tried to set up a .csslintrc file but I can't have csslint load it :/

@pascalchevrel
Copy link
Member Author

I am experimenting with https://www.npmjs.com/package/stylelint and it looks a lot more useful for us than csslint.

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

No branches or pull requests

2 participants