This is a child of #1775842: [meta] Convert all variables to state and/or config systems

CommentFileSizeAuthor
#3 1830148-convert-install_profile_modules-variable-CMI-3.patch1.06 KBcspitzlay
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

cspitzlay’s picture

cspitzlay
Primary language German
Location 🇩🇪🇪🇺
CreditAttribution: cspitzlay commented
Assigned: Unassigned » cspitzlay
cspitzlay’s picture

cspitzlay
Primary language German
Location 🇩🇪🇪🇺
CreditAttribution: cspitzlay commented
Title: Convert 'install_profile_modules' variable to configuration system » Convert 'install_profile_modules' variable to state

That does not look like configuration but like state to me ...

cspitzlay’s picture

cspitzlay
Primary language German
Location 🇩🇪🇪🇺
CreditAttribution: cspitzlay commented
Status: Active » Needs review
FileSize
1830148-convert-install_profile_modules-variable-CMI-3.patch1.06 KB
Berdir’s picture

Berdir
Primary language German
Location Switzerland
CreditAttribution: Berdir commented
Status: Needs review » Reviewed & tested by the community

Looks valid to me.

catch’s picture

catch
he/him
Primary language English
CreditAttribution: catch commented
Status: Reviewed & tested by the community » Fixed

Committed/pushed to 8.x, thanks!

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.