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

Switch is detected as two nesting levels #13

Closed
Rarst opened this issue Dec 17, 2014 · 7 comments
Closed

Switch is detected as two nesting levels #13

Rarst opened this issue Dec 17, 2014 · 7 comments

Comments

@Rarst
Copy link

Rarst commented Dec 17, 2014

PHP switch statement is being detected as two levels of logic nesting. This seems incorrect, unless the intent is to discourage switch altogether.

@TomasVotruba
Copy link
Contributor

Could you send PR with failing test, please? I'll look on that.

@Rarst
Copy link
Author

Rarst commented Nov 12, 2016

Sorry, I am not actively using these sniffs anymore (stopped over version incompatibilities as per other issue you commented on). And as above it wasn't clear what the intent about switch is.

@TomasVotruba
Copy link
Contributor

Ok, thank you for info.

Just personal curiosity: what standards do you use now?

@Rarst
Copy link
Author

Rarst commented Nov 12, 2016

Primarily WordPress CS and Yoast CS downstream from it.

@TomasVotruba
Copy link
Contributor

Thanks for sharing. You use that because of you work with Wordpress or is there something interesting even for non-Wordpress people?

@Rarst
Copy link
Author

Rarst commented Nov 12, 2016

Purely because I work with WordPress, they are specific to it. :)

@TomasVotruba
Copy link
Contributor

I see :)

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

No branches or pull requests

2 participants