Skip to content
This repository has been archived by the owner on May 14, 2021. It is now read-only.

Chef-Solo support #9

Closed
wants to merge 1 commit into from
Closed

Chef-Solo support #9

wants to merge 1 commit into from

Conversation

xrl
Copy link

@xrl xrl commented Jul 12, 2013

I had to patch application_nginx to work with Chef Solo. Added some documentation to explain what I was going for and how to use the changes.

@sethvargo
Copy link
Contributor

Hey @xrl! Thank you for supporting the Opscode Cookbooks! Our process has a couple prerequisites before we can merge your contribution. We need to ensure you've completed a Contributor License Agreement (CLA) and a ticket on our ticket tracker for the release workflow. Pull requests are optional, but should always include the ticket number that they're related to for cross-referencing. Please take a moment to review the below wiki page for the appropriate steps:

Also, it looks like this will not merge cleanly. Could yo please rebase off master?

@cjerdonek
Copy link

This looks like a duplicate of pull request #11 (though that request was filed after this one). That request also has a link to a ticket. I think I prefer the other pull request since it doesn't have any references to Chef::Config[:solo].

@cjerdonek
Copy link

Actually, pull request #8 (which was replaced by #11) precedes this one.

@benlangfeld
Copy link
Contributor

This is a dup of #11 and can be closed.

@benlangfeld
Copy link
Contributor

Does this cookbook need maintainer-ship assistance? I'd be happy to pitch in to get these things moving at a more regular pace.

@sethvargo
Copy link
Contributor

@coderanger would be the one to answer that @benlangfeld

@benlangfeld
Copy link
Contributor

Yup. Unfortunately he seems to be too busy to respond here :(

@coderanger
Copy link
Member

This was already done in #11, as noted above.

@coderanger coderanger closed this Feb 13, 2014
@benlangfeld
Copy link
Contributor

Thanks @coderanger. Just need #11 in a release now as per #16. Momentum 👍

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.

5 participants