Improve how string tokens are dealt with in PregReplaceEModifierSniff. #324

Merged
merged 1 commit into from Jan 3, 2017

Projects

None yet

3 participants

@jrfnl
Contributor
jrfnl commented Jan 1, 2017

Includes additional unit tests.

(Sister-PR to #314 which fixed this for the Mbstring /e sniff)

@jrfnl jrfnl Improve how string tokens are dealt with in PregReplaceEModifierSniff.
Includes additional unit tests.
d1f5199
@coveralls
coveralls commented Jan 1, 2017 edited

Coverage Status

Coverage increased (+0.007%) to 96.772% when pulling d1f5199 on jrfnl:feature/fix-text-string-handling-preg into ca07471 on wimg:master.

@coveralls

Coverage Status

Coverage increased (+0.007%) to 96.772% when pulling d1f5199 on jrfnl:feature/fix-text-string-handling-preg into ca07471 on wimg:master.

@coveralls

Coverage Status

Coverage increased (+0.007%) to 96.772% when pulling d1f5199 on jrfnl:feature/fix-text-string-handling-preg into ca07471 on wimg:master.

@coveralls

Coverage Status

Coverage increased (+0.007%) to 96.772% when pulling d1f5199 on jrfnl:feature/fix-text-string-handling-preg into ca07471 on wimg:master.

@coveralls

Coverage Status

Coverage increased (+0.007%) to 96.772% when pulling d1f5199 on jrfnl:feature/fix-text-string-handling-preg into ca07471 on wimg:master.

@wimg wimg merged commit 4853c6d into wimg:master Jan 3, 2017

3 checks passed

Scrutinizer 1 updated code elements
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
coverage/coveralls Coverage increased (+0.007%) to 96.772%
Details
@jrfnl jrfnl deleted the jrfnl:feature/fix-text-string-handling-preg branch Jan 3, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment