search

acquia / cli

Command-line interface for Acquia Cloud Platform products
https://docs.acquia.com/acquia-cli/
GNU General Public License v2.0
42 stars 47 forks source link

CLI-1272: remote:drush fails with realm alias #1685

Closed danepowell closed 6 months ago

danepowell commented 6 months ago

I let refactoring get the best of me on this one, but it's for the better... not only does this allow several commands to be run with the standard environment prompt, I think this actually fixes a similar bug on ACSF environments with numeric prefixes (01dev). I confirmed that the expected directories in that case are /var/www/site-php/profserv2.01dev, not /var/www/site-php/profserv2.dev as our code and tests previously expected.

codecov[bot] commented 6 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (3006839) 91.27% compared to head (468caf9) 91.08%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #1685 +/- ## ============================================ - Coverage 91.27% 91.08% -0.20% + Complexity 1789 1786 -3 ============================================ Files 122 122 Lines 6375 6368 -7 ============================================ - Hits 5819 5800 -19 - Misses 556 568 +12 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

anavarre commented 6 months ago

This works great for me.

❯ acli remote:drush prod:REDACTED.prod -- status --field=drupal-version
/bin/bash: line 0: cd: /var/www/html/prod:REDACTED.prod/docroot: No such file or directory
9.0.6
❯ ./bin/acli remote:drush prod:REDACTED.prod -- status --field=drupal-version
9.0.6