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

Shouldn't depend on a test cookbook #47

Merged
merged 1 commit into from
Mar 17, 2014
Merged

Conversation

iroller
Copy link
Contributor

@iroller iroller commented Mar 17, 2014

Having this dependency may cause an infinite loop in Berkshelf dependency resolution mentioned here berkshelf/solve#27
This is not the main cause and there is still a bug in berkshelf/solve, but I see no reason of keeping it here.

Having this dependency may cause an infinite loop in Berkshelf dependency resolution mentioned here berkshelf/solve#27
This is not the main cause and there is still a bug in berkshelf/cause, but I see no reason of keeping it here.
@sethvargo
Copy link
Contributor

Yea. This is a real thing. @iroller I just added you as a co-maintainer. My refactor is taking longer than expected 😦. Feel free to merge when ready.

@iroller
Copy link
Contributor Author

iroller commented Mar 17, 2014

Thanks @sethvargo
Do you want me to create a ticket at https://tickets.opscode.com before merging or it's not necessary here?

@sethvargo
Copy link
Contributor

@iroller nope. Merge away 🎈

sethvargo added a commit that referenced this pull request Mar 17, 2014
Shouldn't depend on a test cookbook
@sethvargo sethvargo merged commit fa51f66 into sous-chefs:master Mar 17, 2014
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.

None yet

2 participants