Fabien Potencier

Results 517 comments of Fabien Potencier

@stefan-kamsker Do you have time to have a look at the comments?

Closing as there is no more activities. Feel free to reopen when you have time to address the comments.

Closing as 2 core team members expressed valid concerns about this PR.

@ahmedghanem00 Can you rebase to get rid of the merge commit?

Closing as this out of the scope of the web profiler.

It looks like the tests are still failing.

What about using `{% for value in items %}`?

I'm not sure the use cases are common enough to justify having it in core.