Skip to content

Pr/leafo/627 : Add interpolation support selector#643

Merged
robocoder merged 4 commits intoleafo:masterfrom
Cerdic:PR/leafo/627
Apr 24, 2019
Merged

Pr/leafo/627 : Add interpolation support selector#643
robocoder merged 4 commits intoleafo:masterfrom
Cerdic:PR/leafo/627

Conversation

@Cerdic
Copy link
Contributor

@Cerdic Cerdic commented Apr 24, 2019

I fully reworked the proposal for PR #627 as it had no test case and it appears that the patch was causing @at-root directive to fail in some already existing tests

  • Adding test-case for the interpolation &{#} selector without and with at-root parent
  • keeping the PHPUnit version
  • spliting the patch in several commits to make it more understandable

Tests are OK and PSR2 compliant.

I'm sorry i was not able to keep the credits to @kingyond as I really had do break the full patch in parts to understand how it was (not fully) working before being able to build this new patch, but the original commit really helped me!

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

Successfully merging this pull request may close these issues.

2 participants