Re: [PATCH v2 0/2] doc: git-push: clarify section
- From
Junio C Hamano <gitster@pobox.com>
- Date
- Sep 25, 2025, 15:59 UTC
- Message-ID
- <xmqq1pnuikuf.fsf@gitster.g>
- In-Reply-To
- <pull.1973.v2.git.1758651049.gitgitgadget@gmail.com>
"Julia Evans via GitGitGadget" <gitgitgadget@gmail.com> writes:
Show 16 quoted lines
> I don't love the nested list in PUSH RULES but the sentence starting with > "If the source is a tag or commit object..." is really a tough one to read, > it's not going to be relevant to the vast majority of people, and I think > keeping it contained inside a bullet point will make it much easier to skip > over to get to later information which is more likely to be relevant to > folks. > ... > Changes in v2: > > * Say just "The source can be any object." and don't try to educate folks > further about tags, from Junio's review > * Mention both exceptions to --force working, from Junio's review > * Change "local branch or commit being pushed" => "commit being pushed" to > make it clear that it's just 1 condition, from Junio's review > * Remove an awkward double colon, from Junio's review > * Be explicit that creations are always allowed, from Junio's revew
Nothing questionable stood out during my re-read of these two patches.
Further comments from other reviewers are always welcome, but we find needs for immediate improvements, let's mark the topic for 'next'. Thanks.