[DependencyInjection] Fixed unescaping of class arguments#4707
[DependencyInjection] Fixed unescaping of class arguments#4707avorobiev wants to merge 1 commit intosymfony:masterfrom avorobiev:fix
Conversation
There was a problem hiding this comment.
Could you prepend "[BC BREAK]" please ?
|
@avorobiev could you please take care of the 2 small updates and it will be mergeable. thanks. |
|
Thanks. Just FYI you don't have to create a new PR each time, you can force push to the same branch in order to update the PR. |
|
I don't know it. Thanks. Next time i'll do force push. |
|
If you want to test it you can squash your 2 commits (in 4708) and force push (to have only a single commit) |
|
Yes, @vicb, thank you for tips. |
Commits ------- 036c15e [DependencyInjection] Unescape class arguments part 2 19bdae1 [DependencyInjection] Fixed unescaping of class arguments Discussion ---------- [DependencyInjection] Fixed unescaping of class arguments symfony#4707
#4665 (comment)