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 allow-underscore option for CamelCaseParameterName & CamelCaseVariableName #749
Add allow-underscore option for CamelCaseParameterName & CamelCaseVariableName #749
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice PR,
You created the tests for CamelCaseVariableName but I miss the tests for CamelCaseParameterName. Can you add them?
Sure, I just didn't add them in first place because the whole CamelCaseParameterName tests are missing. Trying to add them today 👍 |
@tvbeek I just added the missing tests, should cover the cases now 🤔 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the PR and the added test!
No problem, thanks for the fast review! |
This replaced the stale PR #363 |
Type: feature
Issue: Resolves #742
Breaking change: no
This PR adds the allow-underscore option to CamelCaseParameterName and CamelCaseVariableName to make it consistent to the other controversial rules. It is used the same way it is used by the existing controversial rules.