Skip to content
This repository has been archived by the owner on Jul 6, 2018. It is now read-only.

added env vars as parameters #14 #15

Merged
merged 2 commits into from
Dec 15, 2014

Conversation

matiasdecarli
Copy link
Contributor

No description provided.

@jkeiser
Copy link
Contributor

jkeiser commented Nov 6, 2014

Can you elaborate a little, perhaps with an example?

@jkeiser jkeiser added this to the 1.0 milestone Nov 6, 2014
@jkeiser
Copy link
Contributor

jkeiser commented Nov 6, 2014

Oh never mind, this is a PR :) Damn waffle. Looking.

@matiasdecarli
Copy link
Contributor Author

No problem. Here are the Docker docs http://docs.docker.com/reference/builder/#env

@jkeiser jkeiser added the Waiting label Nov 6, 2014
@johnewart
Copy link
Contributor

The example in the README doesn't seem to line up, otherwise I think this looks okay, I'll test it out and merge; but it will need to be updated to be mergeable (is that even a word?)

@matiasdecarli
Copy link
Contributor Author

Ok. I will do this tonight.

johnewart added a commit that referenced this pull request Dec 15, 2014
added env vars as parameters #14
@johnewart johnewart merged commit a5f37a0 into chef-boneyard:master Dec 15, 2014
@johnewart johnewart removed the Waiting label Dec 15, 2014
@matiasdecarli
Copy link
Contributor Author

🎉

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

Successfully merging this pull request may close these issues.

4 participants