I did a fresh install of a starter subtheme to test alpha2.

In addition to currently documented install commands, I also ran npm install --global gulp-cli.

Running 'gulp watch' results in this:

#14 {main}
[17:00:54] Finished 'lint:sass' after 1.55 s
[17:00:54] 'drush:cc' errored after 1.64 s
[17:00:54] Error in plugin 'gulp-shell'
Message:
    Command `drush cc css-js` failed with exit code 1
[17:00:55] starter_test.css 82.74 kB
[17:00:55] Finished 'sass' after 2.29 s

There is a larger error output, based on attempting to bootstrap the site. But I am not sure why this command should try to do that at all.

CommentFileSizeAuthor
#6 zurb_foundation-add-drush-alias-for-gulp-2854737-6.patch7.17 KBmariagwyn
#4 zurb_foundation-add-drush-alias-for-gulp-2854737-4.patch5.43 KBmariagwyn
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mariagwyn created an issue. See original summary.

HongPong’s picture

HongPong CreditAttribution: HongPong as a volunteer and at kor group commented

If you are using multi host prefixes it won't work. I had to put in something like this to get it to work in config.js in the child theme:

    alias: {
      css_js: 'drush -l http://ourhost.com cc css-js',

Not sure what the deal is. I wonder if the hyphen is wrong in css-js, it seemed to require that I retyped it?

mariagwyn’s picture

mariagwyn CreditAttribution: mariagwyn as a volunteer commented

This was just a standard install in a repo which is not bootstrapped to run drush at all, though I can with an alias. I am going to recommend that drush be set to 'false' in config.js so that people can turn it on if they actually want it on, which is what the gulpfile.js recommends:

// Defines a task that triggers a Drush cache clear (css-js), you need to edit
// config.js to be able to use this task.

I will have a patch once tested.

mariagwyn’s picture

mariagwyn CreditAttribution: mariagwyn as a volunteer commented
FileSize
zurb_foundation-add-drush-alias-for-gulp-2854737-4.patch5.43 KB

Patch attached which does the following for the STARTER theme:

  1. Disables gulp drush by default:
      drush: {
  enabled: false,
  2. Add drush_alias variable to config.js so alias can be added to each drush call. Variable also added to functions.
  3. Minor cleanup on scss files to reduce gulp warnings.
mariagwyn’s picture

mariagwyn CreditAttribution: mariagwyn as a volunteer commented
Status: Active » Needs review
mariagwyn’s picture

mariagwyn CreditAttribution: mariagwyn as a volunteer commented
FileSize
zurb_foundation-add-drush-alias-for-gulp-2854737-6.patch7.17 KB

Just noting that patch applies cleanly via composer, and updates STARTER as expected. However, I missed committing some changes to the starter.scss file, these are now added on the new patch.

samuel.mortenson’s picture

samuel.mortenson
he/him
Primary language English
CreditAttribution: samuel.mortenson at Acquia commented
Status: Needs review » Fixed

Nice work @mariagwyn!

Status: Fixed » Closed (fixed)

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