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

Add scss-lint to ensure well written Sass #189

Open
verpixelt opened this issue Aug 3, 2015 · 3 comments
Open

Add scss-lint to ensure well written Sass #189

verpixelt opened this issue Aug 3, 2015 · 3 comments
Assignees

Comments

@verpixelt
Copy link
Member

Adding scss-lint together with a Git commit hook could help to keep the Sass files from getting messy.

@zoepage
Copy link
Contributor

zoepage commented Aug 3, 2015

oh this is a good one. thanks :)

@zoepage zoepage self-assigned this Aug 3, 2015
@verpixelt
Copy link
Member Author

I have a implementation of it running in this repo: https://github.com/verpixelt/simple-sass

@zoepage zoepage closed this as completed Aug 5, 2015
@verpixelt
Copy link
Member Author

A Grunt task for Scss lint has been added to the https://github.com/hoodiehq/hoodie-css repo inside the gh-pages branch. For now this task just works manually by typing grunt scsslint (no pre-commit hook). With the ruleset in place right now this task will throw a ton of errors at you. The solution to this is either refactor the whole CSS or adjust the lint rules accordingly (The latter is just a lazy way around the actual problem). This task has to be finalized one way or another.

@verpixelt verpixelt reopened this Aug 12, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants