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

change possibly confusing sample hostname in usage text #133

Merged
merged 3 commits into from
May 16, 2018

Conversation

jjasghar
Copy link

@jjasghar jjasghar commented May 16, 2018

With using myec2node you can imply to someone who does not know much Chef the converge action only works on ec2.

Using web01 is benign enough to say any environment that is a "machine."

@robbkidd robbkidd changed the title Changed myec2node to web01 change possibly confusing sample hostname in usage text May 16, 2018
With using myec2node you can imply to someone who does not know much
Chef the converge action only works on ec2.

Using `web01` is benign enough to say any environment that is a "machine."

Co-authored-by: Robb Kidd <robb@thekidds.org>
Signed-off-by: JJ Asghar <jj@chef.io>
Copy link
Contributor

@tyler-ball tyler-ball left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @jjasghar!

tenor-43559713

Copy link
Contributor

@robbkidd robbkidd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

tenor-180295766

@robbkidd robbkidd merged commit 0d8b441 into master May 16, 2018
@chef-ci chef-ci deleted the jjasghar/helpname branch May 16, 2018 20:23
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.

3 participants