Skip to content

Conversation

@djyou
Copy link
Member

@djyou djyou commented Jan 18, 2018

Fixes #5341


This checklist is used to make sure that common guidelines for a pull request are followed.

General Guidelines

  • The PR has modified HISTORY.rst describing any customer-facing, functional changes. Note that this does not include changes only to help content. (see Modifying change log).

Command Guidelines

  • Each command and parameter has a meaningful description.
  • Each new command has a test.

(see Authoring Command Modules)

@promptws
Copy link

View a preview at https://prompt.ws/r/Azure/azure-cli/5347
This is an experimental preview for @microsoft users.

@djyou
Copy link
Member Author

djyou commented Jan 18, 2018

@yugangw-msft I will rebase once your PR is merged.
/cc @sajayantony @SteveLasker @jluk @yolo3301
Any suggestions on the strings are appreciated.

@jluk
Copy link
Contributor

jluk commented Jan 18, 2018

Thanks for the quick PR @djyou - the error message looks sufficient to me.

Hopefully this mention of error enables users to realize they should use docker-machine or some other machine that can run docker sufficiently.

@yugangw-msft yugangw-msft merged commit 8a0bb2a into Azure:dev Jan 19, 2018
@djyou djyou deleted the doyou/login branch January 22, 2018 19:16
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

Successfully merging this pull request may close these issues.

4 participants