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

PrefixAllGlobals: make independent of case naming conventions #1239

Closed
jrfnl opened this issue Nov 24, 2017 · 0 comments · Fixed by #1278
Closed

PrefixAllGlobals: make independent of case naming conventions #1239

jrfnl opened this issue Nov 24, 2017 · 0 comments · Fixed by #1278

Comments

@jrfnl
Copy link
Member

jrfnl commented Nov 24, 2017

I think the PrefixAllGlobals sniff should be more code style independent, i.e. remove the need for an underscore between the prefix and the class/function/variable name.

Should the sniff really care about whether people use Camel_Caps, camelCase, snake_case or any other variant of naming conventions, as long as whatever name is used, is prefixed ?

Just wondering what the opinions are about this.

Related to #763

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
1 participant