From: Daniele Sassoli <danielesassoli@gmail.com>
To: Junio C Hamano <gitster@pobox.com>,
Daniele Sassoli via GitGitGadget <gitgitgadget@gmail.com>
Cc: git@vger.kernel.org
Subject: Re: [PATCH] doc:clarify which remotes can be used when contributing
Date: Wed, 20 Aug 2025 15:07:36 +0100 [thread overview]
Message-ID: <363ac6d1-8444-4d48-a66b-51ea139f4e87@gmail.com> (raw)
In-Reply-To: <xmqqms7vc8mo.fsf@gitster.g>
On 19/08/2025 22:19, Junio C Hamano wrote:
> "Daniele Sassoli via GitGitGadget" <gitgitgadget@gmail.com> writes:
>> https://github.com/gitgitgadget/git and open a PR either with the "New pull
>> request" button or the convenient "Compare & pull request" button that may
>> appear with the name of your newly pushed branch.
>> +If you're using https://github.com/git/git as your remote, you will need to
>> +open the pull-request from your fork, selecting `git/git` as base.
>> +
>> +The differences between using `gitgitgadget/git` and `git/git` as your base can
>> +be found [here](https://gitgitgadget.github.io/#should-i-use-gitgitgadget-on-gitgitgadgets-git-fork-or-on-gits-github-mirror)
> Looking at the table, there is no advantage to use git/git at all.
>
> Instead of telling them that they can use either (with reduced
> capabilities if you pick one of them instead of the other), wouldn't
> it be easier for the user if this section taught them how to switch
> their fork that they originally created out of git/git to be based
Most of the document, including the "Getting Started" section, points to cloning
from git/git. It's only when it comes to the gitgitgadget section that we
mention gitgitgadget/git.
It's true that there are no advantages of using git/git over gitgitgadget/git,
but I would argue that the disadvantages are quite minor and definitely don't
impact someone at their first contribution?
As we've told the reader so far to use git/git, I don't see why we should tell
him to change his remote for potentially no benefit. If we're saying that
gitgitgadget should be used only on gitgitgadget/git, then we should restructure
the document to point to that from the beginning, no?
PS. please note it's my first time replying on the mailing list directly,
apologies if I did something wrong, like wrong posting style or wrong people in
cc.
> on gitgitgadget/git instead? Something along the lines of
>
> ... If you originally forked from https://github.com/git/git/,
> you can easily correct it by running (you only need to do this
> once):
>
> $ git remote set-url origin https://github.com/gitgitgadget/git/
> $ git fetch --prune origin
>
> A pull request at https://github.com/gitgitgadget/git/ can be
> opened once you do so.
>
> but you'd need to validate the procedure, as I didn't try it myself.
>
> Thanks.
next prev parent reply other threads:[~2025-08-20 14:07 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-08-19 19:14 [PATCH] doc:clarify which remotes can be used when contributing Daniele Sassoli via GitGitGadget
2025-08-19 21:19 ` Junio C Hamano
2025-08-20 14:07 ` Daniele Sassoli [this message]
2025-08-20 21:16 ` Junio C Hamano
2025-08-22 9:14 ` Daniele Sassoli
2025-08-22 17:56 ` Junio C Hamano
2025-08-22 18:45 ` Elijah Newren
2025-08-23 9:12 ` [PATCH v2] " Daniele Sassoli via GitGitGadget
2025-08-25 15:39 ` Elijah Newren
2025-08-25 16:21 ` Junio C Hamano
2025-08-26 6:50 ` Daniele Sassoli
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=363ac6d1-8444-4d48-a66b-51ea139f4e87@gmail.com \
--to=danielesassoli@gmail.com \
--cc=git@vger.kernel.org \
--cc=gitgitgadget@gmail.com \
--cc=gitster@pobox.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).