New command for deploying from this repo #1

Merged
merged 2 commits into from Jul 1, 2013

Conversation

Projects
None yet
2 participants
@santiago26
Contributor

santiago26 commented Jun 28, 2013

Cloning using https more reliable. Application can also be created with a single command.

New command for deploying from this repo
Cloning using https more reliable. Application can also be created with a single command.
@gfidente

This comment has been minimized.

Show comment Hide comment
@gfidente

gfidente Jul 1, 2013

Owner

Hi there and thanks for helping.

I agree with the idea of documenting the --from-code option but have a couple of comments around the pull request.

Maybe we should replace "Get the source from the cloud" with a more descriptive paragraph? How about just a few words explaining the code gets cloned on the servers automatically?

It looks like one doesn't need to use "rhc git-clone" as the application is cloned locally after the creation process completes. Any particular reason for adding that?

Owner

gfidente commented Jul 1, 2013

Hi there and thanks for helping.

I agree with the idea of documenting the --from-code option but have a couple of comments around the pull request.

Maybe we should replace "Get the source from the cloud" with a more descriptive paragraph? How about just a few words explaining the code gets cloned on the servers automatically?

It looks like one doesn't need to use "rhc git-clone" as the application is cloned locally after the creation process completes. Any particular reason for adding that?

Update README.creole
removed command git-clone, added paragraph about code
@santiago26

This comment has been minimized.

Show comment Hide comment
@santiago26

santiago26 Jul 1, 2013

Contributor

Yes, it's a mistake. Git-clone is not needed, you are right. Fixed.

Contributor

santiago26 commented Jul 1, 2013

Yes, it's a mistake. Git-clone is not needed, you are right. Fixed.

@gfidente

This comment has been minimized.

Show comment Hide comment
@gfidente

gfidente Jul 1, 2013

Owner

thanks!

Owner

gfidente commented Jul 1, 2013

thanks!

@gfidente gfidente closed this Jul 1, 2013

@gfidente gfidente reopened this Jul 1, 2013

gfidente added a commit that referenced this pull request Jul 1, 2013

Merge pull request #1 from santiago26/patch-1
New command for deploying from this repo

@gfidente gfidente merged commit 025a3aa into gfidente:master Jul 1, 2013

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