-
Notifications
You must be signed in to change notification settings - Fork 2k
Fix issue #526: There is no *adding* remote repositories in previous sections. #528
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
Conversation
Nice, but could I suggest a tweak?
|
@ben Excellent. 😃 |
Question: could you please tell me what |
@YueLinHo this is just to insert an index entry reference at this place in the book. At the end of the book, there is a term index and you will find this reference under the index term "git commands", sub term "remote". This is present in the pdf only. |
@jnavila Thank you so much. 😄 |
Rebased! |
|
||
We've mentioned and given some demonstrations of adding remote repositories in previous sections, but here is how to do it explicitly.(((git commands, remote))) | ||
We've mentioned and given some demonstrations of 'clone' command which implicitly adds the `origin` remote for you in previous sections. | ||
Here's how to add one explicitly.(((git commands, remote))) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Close. How about this:
We've mentioned and given some demonstrations of how the `clone` command implicitly adds the `origin` remote for you.
Here's how to add a new remote explicitly.(((git commands, remote)))
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes, it's better. Nice!!!
…vious sections. There is no `git add remote`, related command, or statement in previous sections, but `git clone`. And the only remote repository mentioned is "origin" which is added implicitly via clone command. Fix the sentence for that.
@ben I have rebased and updated this PR. Please review it, again. 😃 |
Nice. Thanks! |
Fix issue #526: There is no *adding* remote repositories in previous sections.
@ben thank you. 😃 |
There is no
git add remote
, related command, or statement in previous sections, butgit clone
.And the only remote repository mentioned is "origin" which is added automatically via clone command.
Fix the sentence for that.
Only one question: the words
directly
vs.explicitly
. Which one is better?