Skip to content
This repository was archived by the owner on Mar 5, 2025. It is now read-only.

blt should have an acsf:init:drush command #779

Closed
nikgregory opened this issue Dec 9, 2016 · 2 comments
Closed

blt should have an acsf:init:drush command #779

nikgregory opened this issue Dec 9, 2016 · 2 comments
Assignees
Labels
Enhancement A feature or feature request

Comments

@nikgregory
Copy link
Contributor

Since a lot of the blt action will do a composer update, acsf:init can be overkill once executed. But when acsf is updated, the drush commands need to be run or the factory will not deploy the code.

if we add a drush only command, we can let composer deal with the update and make sure the proper files are in place

<target name="acsf:init:drush" description="Refreshes the acsf settings and hook files.">
    <echo>Executing initialization command for acsf module.</echo>
    <drush command="acsf-init">
      <option name="include">"${docroot}/modules/contrib/acsf/acsf_init"</option>
    </drush>
    <echo>Please add acsf_init as a dependency for your installation profile to ensure that it remains enabled.</echo>
    <echo>An example alias file for ACSF is located in /drush/site-aliases/example.acsf.aliases.drushrc.php.</echo>
  </target>
@nikgregory nikgregory added the Enhancement A feature or feature request label Dec 9, 2016
@nikgregory nikgregory self-assigned this Dec 9, 2016
@nikgregory
Copy link
Contributor Author

PR #782

@grasmash
Copy link
Contributor

Merged #782

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Enhancement A feature or feature request
Projects
None yet
Development

No branches or pull requests

2 participants