Skip to content

feat: add aggregate_gc_threshold of 0 to drupal12 settings.ddev.php (#8215) [skip ci]#8215

Merged
rfay merged 1 commit into
ddev:mainfrom
rpkoller:20260310-rpkoller-asset-aggregates-d12
Mar 27, 2026
Merged

feat: add aggregate_gc_threshold of 0 to drupal12 settings.ddev.php (#8215) [skip ci]#8215
rfay merged 1 commit into
ddev:mainfrom
rpkoller:20260310-rpkoller-asset-aggregates-d12

Conversation

@rpkoller
Copy link
Copy Markdown
Collaborator

The Issue

Set the newly introduced aggregate_gc_threshold to 0 (https://www.drupal.org/project/drupal/issues/3505776) in settings.ddev.php for drupal12

the comment describing the newly added setting could be improved i suppose

  • Fixes #REPLACE_ME_WITH_RELATED_ISSUE_NUMBER

How This PR Solves The Issue

Manual Testing Instructions

Automated Testing Overview

Release/Deployment Notes

@rpkoller rpkoller requested a review from a team as a code owner March 10, 2026 18:53
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Mar 10, 2026

Copy link
Copy Markdown
Member

@rfay rfay left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The only thing really required here is consensus in Drupal folk that this is the right setting for local dev.

// https://www.drupal.org/forum/support/post-installation/2018-07-18/enable-drupal-8-backend-errorlogdebugging-mode
$config['system.logging']['error_level'] = 'verbose';

// Omit to retain asset aggregates
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The comment is a bit misleading; people shouldn't be editing this file. Having a reference to the issue is fine.

@rfay rfay force-pushed the 20260310-rpkoller-asset-aggregates-d12 branch from 92e67ad to c5ba18a Compare March 23, 2026 14:15
@rfay
Copy link
Copy Markdown
Member

rfay commented Mar 27, 2026

I'll go ahead and pull this @rpkoller but if you can make sure that some of your colleagues agree that it belongs in local dev by default it will be much appreciated.

@rfay rfay changed the title feat: add aggregate_gc_threshold of 0 to drupal12 settings.ddev.php feat: add aggregate_gc_threshold of 0 to drupal12 settings.ddev.php (#8215) [skip ci] Mar 27, 2026
@rfay rfay merged commit 736187d into ddev:main Mar 27, 2026
36 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants