Add CSS linting by guillaumevincent · Pull Request #72 · patternfly/patternfly-css · GitHub
Skip to content

Add CSS linting#72

Open
guillaumevincent wants to merge 1 commit into
patternfly:masterfrom
guillaumevincent:master
Open

Add CSS linting#72
guillaumevincent wants to merge 1 commit into
patternfly:masterfrom
guillaumevincent:master

Conversation

@guillaumevincent

Copy link
Copy Markdown

Install ruby scss-lint package.
Use the same mechanism as bootstrap.

Fixes https://patternfly.atlassian.net/browse/PTNFLY-2422

@andresgalante

Copy link
Copy Markdown
Member

@guillaumevincent

guillaumevincent commented Sep 11, 2017

Copy link
Copy Markdown
Author

I started small, and will tackle one feature at a time
I think travis is not enable right?

The current test (npm test) will raise an error because of the errors in sources/scss/*.scss

@andresgalante

Copy link
Copy Markdown
Member

@bleathem this one works for me, can you please review it?

@guillaumevincent thanks a lot, I love it!

@andresgalante

Copy link
Copy Markdown
Member

@guillaumevincent sorry that it took me so long to loop back to this. Bootstrap 4 beta 2 changed to stylelint and I think it's a good move. What do you think? should we follow it and merge a linter here?

thanks a lot!

@guillaumevincent

Copy link
Copy Markdown
Author

let me update the pull request @andresgalante

Install ruby scss-lint package.
Use the same mechanism as bootstrap.
@guillaumevincent

Copy link
Copy Markdown
Author

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.

3 participants