- π©πͺGermany geek-merlin Freiburg, Germany
And a much bigger objection: The MR contains @dawehner's quich shot from #58 from 7 years ago, and is a strong Drupalism.
Let's instead leverage the
kernel.environment
container parameter. It is symfony standard and already heavily supported in our Drupal kernel:
- \Drupal\Core\DrupalKernel::getKernelParameters
- \Drupal\Core\DrupalKernel::getContainerCacheKey - π©πͺGermany geek-merlin Freiburg, Germany
One objection to a setting: IIRC, settings jump in quite late. A container parameter would be better in this regard.
- @quietone opened merge request.
- First commit to issue fork.