search

computerminds / cm_config_tools

DEPRECATED: Mirror of http://cgit.drupalcode.org/cm_config_tools
1 stars 0 forks source link

Remove reverting command? #27

Closed anotherjames closed 7 years ago

anotherjames commented 7 years ago

config_update now has its own drush command for reverting config. It's not quite as 'nice', because there's one for a single revert, another for multiple reverts, and neither will work for config that is yet to import (they have a separate command for that).

I think because of that, we'll keep ours, as it's useful in update hooks to just blindly apply config changes regardless of whether it existed before or not.

So I'll immediately close this issue, but it at least exists to document my thought process ;-)

anotherjames commented 7 years ago

To be fair, there is a difference between the drush commands and the methods on the relevant services. So we could remove our drush command whilst keeping the revert method on our service, as that's the useful bit for update hooks. :man_shrugging: