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 function-name-case #3674

Merged
merged 2 commits into from Sep 20, 2018

Conversation

3 participants
@vankop
Contributor

vankop commented Sep 16, 2018

Which issue, if any, is this issue related to?

"Closes #3658"

Is there anything in the PR that needs further explanation?

"No, it's self explanatory."

@ota-meshi

@vankop Thank you for this PR!
I have some change requests.

Show resolved Hide resolved lib/rules/function-name-case/__tests__/index.js
Show resolved Hide resolved lib/rules/function-name-case/index.js Outdated
Show resolved Hide resolved lib/rules/function-name-case/index.js Outdated
@vankop

This comment has been minimized.

Contributor

vankop commented Sep 19, 2018

@ota-meshi updated

@ota-meshi

LGTM!

@ntwb

ntwb approved these changes Sep 20, 2018

@ntwb ntwb merged commit 5961189 into stylelint:master Sep 20, 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.01%) to 96.372%
Details
@ntwb

This comment has been minimized.

Member

ntwb commented Sep 20, 2018

Changelog: 2464149

Added: function-name-case autofix (#3674).

@ntwb

This comment has been minimized.

Member

ntwb commented Sep 20, 2018

Thanks @vankop, keep your eyes out for an invite to the stylelint organisation :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment