Problem/Motivation

Lets add theme support.

Proposed resolution

Just expand the support in the drush task.

Remaining tasks

  1. Is $extension the right name for a variable that describes a module and a theme?
  2. Should we rename the cd-em command since it stands for config-devel-export-module ?

User interface changes

n/a

API changes

Addition.

Data model changes

n/a

CommentFileSizeAuthor
#4 2580315-4.patch3.62 KBbenjy
config-devel-theme-support.patch1.67 KBbenjy
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

benjy created an issue. See original summary.

chx’s picture

chx CreditAttribution: chx commented

Perhaps rename to cd-ee but add an alias cd-em to make sure existing scripts dont break

benjy’s picture

benjy CreditAttribution: benjy at PreviousNext commented

How do you feel about simply drush cde? For Config Devel Export

benjy’s picture

benjy CreditAttribution: benjy at PreviousNext commented
FileSize
2580315-4.patch3.62 KB

Re-rolled for the optional configuration issue and renamed command to config-devel-export, added a cde alias and preserved the existing alias.

chx’s picture

chx CreditAttribution: chx commented
Status: Needs review » Reviewed & tested by the community
benjy’s picture

benjy CreditAttribution: benjy at PreviousNext commented
Status: Reviewed & tested by the community » Fixed

Committed.

  • benjy committed d4cdd21 on 8.x-1.x 
    Issue #2580315 by benjy: Add support for themes

Status: Fixed » Closed (fixed)

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