It's not the end of the world as it is, but I want to set a high bar for DX
if possible. The stuff you added for interactive use is really awesome in
that regard. ;)

In general, when we have commands that are dependent on additional system
config we should try and point people in the right direction if they're not
set up to use it yet, rather than things fail with system default error
messages.

If you don't mind adding an additional check and a more n00b friendly
message, that would be much appreciated.


On Mon, Sep 16, 2013 at 5:55 PM, Mike Milano <notifications@github.com>wrote:

> In terminus.drush.inc:
>
> > @@ -1093,6 +1104,62 @@ function drush_terminus_pantheon_site_tunnel_close($pid = NULL) {
> >  }
> >
> >  /**
> > + * Mounts the appserver for a specific environment.
> > + */
> > +function drush_terminus_pantheon_site_mount($site_uuid = FALSE, $environment = FALSE, $destination = FALSE) {
> > +  $session_data = terminus_bootstrap();
>
> I thought about that, but the output is printed in case of an error, and
> the error is very clear. It looks like this:
>
> Unable to mount: sh: sshfs: command not found                                                                            [error]
>
> Wouldn't be a problem though, let me know if you still want that after
> this is considered.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/pantheon-systems/terminus/pull/34/files#r6394322>
> .
>



-- 

Josh Koenig
Co-Founder, Pantheon Systems Inc
josh@getpantheon.com