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 autofix to unit-case rule #3725

Merged
merged 3 commits into from Oct 17, 2018

Conversation

3 participants
@yepninja
Member

yepninja commented Oct 16, 2018

Closes #3664

@jeddy3

@yepninja This PR is looking great too!

I've made one minor request.

This PR supersedes the stalled #3160 and #3080 as it makes use of the value parser.

}
// value = value.replace(/\*/g, ",");
const errors = [];

This comment has been minimized.

@jeddy3

jeddy3 Oct 17, 2018

Member

As with #3720 (comment), can we name this variable violations please?

This comment has been minimized.

@yepninja
@jeddy3

jeddy3 approved these changes Oct 17, 2018

Thanks, LGTM

@jeddy3 jeddy3 merged commit 62feb43 into stylelint:master Oct 17, 2018

3 checks passed

continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
coverage/coveralls Coverage increased (+0.02%) to 96.297%
Details

This was referenced Oct 17, 2018

@jeddy3

This comment has been minimized.

Member

jeddy3 commented Oct 17, 2018

  • Added: unit-case autofix (#3725).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment