Skip to content

Conversation

@KrishMunot
Copy link
Contributor

No description provided.

@kmod
Copy link
Collaborator

kmod commented May 6, 2016

This looks great, but would you mind adding some sort of doc on how to use it?

@KrishMunot
Copy link
Contributor Author

Yes, I will add it soon.
Can you merge this while we are at it?
Thanks! 😃

@corona10
Copy link

corona10 commented May 7, 2016

@kmod It looks like same docker code when I made PR as #1099 but it looks good. :-)

@kmod
Copy link
Collaborator

kmod commented May 9, 2016

Hi @KrishMunot, would you mind cherry-picking corona10's original commit, rather than copying the file and creating a new commit? You should be able to do it with

git fetch origin eae40ee9
git cherry-pick eae40ee9

Could you also add a small doc somewhere? I'm pretty hesitant to merge something like this without at least a tiny amount of documentation, since otherwise we/I won't remember what's going on with it and it is likely to just get removed. Even just corona10's two lines of instruction from #1099 would be great to have in the readme.

@kmod
Copy link
Collaborator

kmod commented Jun 8, 2016

I'm going to close this and we can keep discussing on #1099

@kmod kmod closed this Jun 8, 2016
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.

3 participants