-
Notifications
You must be signed in to change notification settings - Fork 2
Description
Up until now we relied on settings overriddes to disable aggregation locally:
annertech-ddev/settings.local.devmode.php
Line 76 in 8bc95a1
| $config['system.performance']['css']['preprocess'] = FALSE; |
Lately we switched to the new drush theme:dev on command (drush 13.6+) with the idea that we move away from custom overrides towards a community driven approach.
annertech-ddev/commands/host/devmode
Line 23 in 8bc95a1
| if [[ $DRUSH_IS_COMPATIBLE = true ]]; then ddev exec drush theme:dev off; fi |
Since then we are experiencing a lot of commits where disabling aggregation has sneaked in config changes, something we must avoid.
I strongly suspect the switch to drush is the issue here, as it changes the values in Drupal and doesn't override them.
We need to consider if we revert to custom overrides or if we strengthen our pre-commit and pre-push hooks to avoid this reaching production sites.