Don't error for syntax errors in StrictInArray sniff #772

Merged
merged 1 commit into from Jan 6, 2017

Projects

None yet

4 participants

@JDGrimes
Contributor
JDGrimes commented Jan 5, 2017

This should be left up to the Generic.PHP.Syntax sniff.

See #770 (comment)

@JDGrimes JDGrimes added this to the 0.11.0 milestone Jan 5, 2017
@grappler
Contributor
grappler commented Jan 5, 2017

@JDGrimes Did you see that you need to update the tests also?

@JDGrimes @JDGrimes JDGrimes Don't error for syntax errors
ce37414
@JDGrimes
Contributor
JDGrimes commented Jan 5, 2017

@grappler Yep, fixed now.

@jrfnl
jrfnl approved these changes Jan 5, 2017 View changes
@GaryJones GaryJones merged commit ddee36c into develop Jan 6, 2017

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
@GaryJones GaryJones deleted the no-syntax-error-errors branch Jan 6, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment