Skip to content

Conversation

td2014
Copy link
Contributor

@td2014 td2014 commented Aug 21, 2017

As I was reading section 10.2 regarding git internals and tree creation, I found what I think might be a minor omission of an "--add" flag and a superfluous "update-index" statement. I provide what I believe is a more streamlined and corrected version in this PR. Thanks.

@td2014
Copy link
Contributor Author

td2014 commented Oct 9, 2017

Added some additional information in 5.2 (Distributed Git - Contributing to a Project), to explain the format of the return message from the "git push" operation. Namely, a brief explanation of the .. fromref -> toref. I think this might be helpful to understand to rest of the discussion in the section. Thanks.

@td2014 td2014 changed the title Minor cleanup in 10.2 regarding new tree creation Minor cleanup in 10.2 regarding new tree creation and in 5.2 to explain format of return message from push. Oct 9, 2017
The last line of the output above shows a useful return message from the push operation.
The basic format is `<oldref>..<newref> fromref \-> toref`, where `oldref` means the old reference, `newref` means the new reference, `fromref` is the name of the local reference being pushed, and `toref` is the name of the remote reference being updated.
You'll see similar output like this below in the discussions, so having a basic idea of the meaning will help in understanding the various states of the repositories.
More details are available in the documentation for https://git-scm.com/docs/git-push[git push].
Copy link
Member

Choose a reason for hiding this comment

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

Can you change the link text to "git-push"? That's how you'd get the man page.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@ben . Your requested change has been made. Thanks.

@ben ben merged commit bd540ab into progit:master Oct 9, 2017
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