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 example issue number to kitematic contributors guide instead of… #50

Merged
merged 1 commit into from
Dec 29, 2015
Merged

added example issue number to kitematic contributors guide instead of… #50

merged 1 commit into from
Dec 29, 2015

Conversation

londoncalling
Copy link
Contributor

… placeholders, and copyedit fixes

Signed-off-by: Victoria Bialas victoria.bialas@docker.com

@londoncalling
Copy link
Contributor Author

@moxiegirl @FrenchBen I updated the online docs version of the Kitematic Contirbutors' Guide w/some copy-edits, and added the issue # we used for the example (same as the Gist http://git.io/v8d55), can you guys take a quick look at let me know if it's good to go? Thanks! Vicky

$ git push origin xxxx-branch:xxxx-branch

where _xxxx_ is the number of the issue your worked on.
$ git push origin 1191-branch

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this needs to be git push origin 1191-branch:1191-branch in order to create the remote branch with the same name as the local branch.
The above only works if the remote branch exists and has the same name

…de create of new branch, and references to repo name now docker/kitematic

Signed-off-by: Victoria Bialas <victoria.bialas@docker.com>
@londoncalling
Copy link
Contributor Author

@FrenchBen Thanks! I think I got them all. Let me know, thanks for the quick review.

@FrenchBen
Copy link

LGTM

londoncalling added a commit that referenced this pull request Dec 29, 2015
added example issue number to kitematic contributors guide instead of…

Merging this, @moxiegirl I got all Ben's comments in, and re-ran make docs to check it. LGTM 👍 :-)
@londoncalling londoncalling merged commit eb2e88f into docker:master Dec 29, 2015
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.

2 participants