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

Feature: all variables must be camel case - strict #42

Merged
merged 5 commits into from
Oct 25, 2019

Conversation

michalbundyra
Copy link
Member

Defined variables, variables used in strings and class properties must
be now camelCased (strict) - so two capital letters next to each other
are not allowed

Disable PSR2.Classes.PropertyDeclaration.Underscore as it is now covered
in Webimpress\NamingConventions\ValidVariableName sniff.

Defined variables, variables used in strings and class properties must
be now camelCased (strict) - so two capital letters next to each other
are not allowed

Disable `PSR2.Classes.PropertyDeclaration.Underscore` as it is now covered
in `Webimpress\NamingConventions\ValidVariableName` sniff.
@michalbundyra michalbundyra added the enhancement New feature or request label Oct 21, 2019
@michalbundyra michalbundyra added this to the 1.1.0 milestone Oct 21, 2019
Method names are validated by PSR-2 sniff which doesn't require strict
camelCase names. There is no any option to change it to strict as well.
michalbundyra added a commit that referenced this pull request Oct 25, 2019
Feature: all variables must be camel case - strict
michalbundyra added a commit that referenced this pull request Oct 25, 2019
@michalbundyra michalbundyra merged commit f8afacf into develop Oct 25, 2019
michalbundyra added a commit that referenced this pull request Oct 25, 2019
@michalbundyra michalbundyra deleted the feature/camel-case-variables branch October 25, 2019 20:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant