Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixes #4500 to resolve config split recipe issues and #4501 for deprecated Twig code. #4502

Merged
merged 6 commits into from
Apr 20, 2022

Conversation

mikemadison13
Copy link
Contributor

Motivation
The config split recipe has a few gaps (which I have to fix manually all the time) and some deprecated code. See #4500 and #4501.

Proposed changes

  • recipe creates .htaccess files for all splits
  • split templates are not "off" by default (status to false)
  • recipe enables config split and config filter by default in core.extension
  • expanded tests
  • replaced deprecated Twig_Loader_Filesystem and Twig_Environment. see https://twig.symfony.com/doc/2.x/deprecated.html#environment

@mikemadison13 mikemadison13 added Bug Something isn't working Enhancement A feature or feature request labels Apr 12, 2022
@danepowell danepowell enabled auto-merge (squash) April 20, 2022 14:40
@danepowell danepowell merged commit f61aa44 into acquia:main Apr 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something isn't working Enhancement A feature or feature request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants