Nicolas Grekas
Nicolas Grekas
Oh, I completely misunderstood the PR. Makes sense to me. Just one issue: when renderIcon fails, we should fallback to regular runtime-resolution IMHO.
This should target 8.1 since it's a new feature (not a bug fix)
Well, we do have recipes attached to packs, see https://github.com/symfony/recipes/tree/main/symfony/webapp-pack/1.0 This allows prioritizing some more specific config files when we know some deps are installed together (doctrine-messenger eg here). Note...
What about improving the HtmlErrorRendered instead so that it dumps a text version after the HTML one in an HTML comment? (we'd just need to ensure the dump cannot break...
Solving this in a bundle looks appropriate to me. DI is the proper way to improve the DX here. Let's keep things simple.