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

fix #136 thanks @jeremymv2 #141

Merged
merged 1 commit into from Nov 2, 2016

Conversation

Projects
None yet
3 participants
@chris-rock
Collaborator

chris-rock commented Nov 2, 2016

No description provided.

@binamov binamov added the in progress label Nov 2, 2016

@@ -5,6 +5,26 @@ This directory contains a simple vagrant setup that expects you have a Chef Serv
1.) Upload cookbook to Chef Server
```

This comment has been minimized.

@jeremymv2

jeremymv2 Nov 2, 2016

Contributor

You may just want a comment like "From within the cookbook you can run the following commands:" type of thing so they know berks is run from command prompt inside the cookbook directory.

@jeremymv2

This comment has been minimized.

Contributor

jeremymv2 commented Nov 2, 2016

LGTM 👍

fix #136 thanks @jeremymv2
Signed-off-by: Christoph Hartmann <chris@lollyrock.com>

@chris-rock chris-rock merged commit 7a70d6f into master Nov 2, 2016

2 checks passed

DCO This commit has a DCO Signed-off-by
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

@chris-rock chris-rock deleted the chris-rock/fix-136 branch Nov 2, 2016

@binamov binamov removed the in progress label Nov 2, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment