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 stylelint #10

Merged
merged 1 commit into from Jul 12, 2018

Conversation

@ahawkins
Copy link
Contributor

commented Jul 11, 2018

This PR adds stylelint. No prior configuration is required. The standard configuration file is used if stylelint/config.json is not found in the configuration repo. Tests files matching **/*.{css,scss,less,sss}.

@ahawkins

This comment has been minimized.

Copy link
Contributor Author

commented Jul 11, 2018

@avaly does this work for you? I'm not sure if the default glob makes sense for all projects, or if there needs to be a feature to customize the glob. If the default glob is too general, then it can be ignored with .stylelintignore in each project.

@avaly

This comment has been minimized.

Copy link

commented Jul 11, 2018

The default glob should work for most projects.

@ahawkins ahawkins force-pushed the feature/stylelint branch from 81dcb16 to 9571020 Jul 12, 2018

@ahawkins ahawkins merged commit 0a33ed7 into master Jul 12, 2018

5 of 8 checks passed

editorconfig .editorconfig covers files in source control
Details
eslint eslint - Javascript/JSON linting
Details
phpcs PHP CodeSniffer - lint php code
Details
custom Custom checks
Details
gometalinter gometalinter - Aggregate multiple go linters
Details
rubocop Syntax Check
Details
shellcheck Shellcheck - static analysis tool for shell scripts
Details
syntax Syntax Check
Details

@ahawkins ahawkins deleted the feature/stylelint branch Jul 12, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.