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

added fix for chef-zero wrapper for chef 10 to capture exit codes #356

Closed
wants to merge 1 commit into from

Conversation

kamalim
Copy link

@kamalim kamalim commented Feb 11, 2014

No description provided.

@sethvargo
Copy link
Contributor

Isn't this the same #355?

@kamalim
Copy link
Author

kamalim commented Feb 11, 2014

Hello:

Its the same.But the build is failing for both.Can you please help fixing
it.
We have tested it locally and it works

Thanks
Kamalika

On Tue, Feb 11, 2014 at 9:58 PM, Seth Vargo notifications@github.comwrote:

Isn't this the same #355#355

Reply to this email directly or view it on GitHubhttps://github.com//pull/356#issuecomment-34771770
.

@fnichol
Copy link
Contributor

fnichol commented Feb 12, 2014

Hey there @kamalim, the failing on Travis is entirely my fault (a newer version of minitest that needs a pin for now) which is fixed in master. I believe with #353 we should be good here.

Thank you so much!

@fnichol fnichol closed this Feb 12, 2014
@test-kitchen test-kitchen locked and limited conversation to collaborators Nov 16, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants