Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

BLT Sync Commands Don't Respect -y #1853

Closed
mikemadison13 opened this issue Jul 28, 2017 · 3 comments
Closed

BLT Sync Commands Don't Respect -y #1853

mikemadison13 opened this issue Jul 28, 2017 · 3 comments

Comments

@mikemadison13
Copy link
Contributor

My system information:

  • Operating system type: Mac OS
  • Operating system version: 10.11.6
  • BLT version: 8.9.0

When I run this command:

blt sync:refresh -y
blt sync:db -y

I get the following output:

blt sync:db -y
Executing command blt sync:db -y --define drush.alias=self inside of Drupal VM...
[warning] The xDebug extension is loaded. This will significantly decrease performance.
> setup:settings
[Acquia\Blt\Robo\Tasks\DrushTask] Running /var/www/<project>/vendor/bin/drush cache-clear drush --uri=default && /var/www/<project>/vendor/bin/drush sql-drop --uri=default && /var/www/<project>/vendor/bin/drush sql-sync @<project>.test @<project>.local --structure-tables-key=lightweight --create-db --sanitize --uri=default && /var/www/<project>/vendor/bin/drush cache-clear drush --uri=default in /var/www/<project>/docroot
'drush' cache was cleared.                                                                                                        [success]
Do you really want to drop all tables in the database drupal? (y/n):

And I expected this to happen:
I would expect the -y to automatically clear each question during the sync for dropping the db, but it does not.

@dpagini
Copy link
Contributor

dpagini commented Jul 28, 2017

Looks like a dupe of #1841. Slightly different request I think, but same root problem.

@mikemadison13
Copy link
Contributor Author

@dpagini yeah it does look like a dupe, i'm flexible in how this is approached. I just remember @grasmash saying that it "should" accept a -y (and blt itself indicates that). So i'm not sure if we want to fix both of this issues and assume try AND accept a -y / -n or not.

@grasmash
Copy link
Contributor

I consider this addressed by #1841.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants