From: Junio C Hamano <gitster@pobox.com>
To: "Torsten Bögershausen" <tboegi@web.de>
Cc: Alexander Mills <alexander.d.mills@gmail.com>, git@vger.kernel.org
Subject: Re: please add link / url to remote - when - git push
Date: Mon, 30 Oct 2023 15:55:56 +0900 [thread overview]
Message-ID: <xmqqa5s0egzn.fsf@gitster.g> (raw)
In-Reply-To: <20231030063633.GA7451@tb-raspi4> ("Torsten Bögershausen"'s message of "Mon, 30 Oct 2023 07:36:33 +0100")
Torsten Bögershausen <tboegi@web.de> writes:
> On Sun, Oct 29, 2023 at 06:15:35PM -0500, Alexander Mills wrote:
>> When a feature branch is pushed, we get a link in the console to go to
>> remote, however when I push directly to main/master, no such link, eg:
>>
>> ```
>> alex.mills@alex node-be % git push
>> Enumerating objects: 20, done.
>> Counting objects: 100% (20/20), done.
>> Delta compression using up to 12 threads
>> Compressing objects: 100% (10/10), done.
>> Writing objects: 100% (11/11), 1.56 KiB | 799.00 KiB/s, done.
>> Total 11 (delta 7), reused 0 (delta 0), pack-reused 0
>> remote: Resolving deltas: 100% (7/7), completed with 7 local objects.
>> remote: Bypassed rule violations for refs/heads/main:
>> remote:
>> remote: - Changes must be made through a pull request.
>> remote:
>> To github.com:elx-onlinx/beautychat-chatcards.git
>> ffe1e05..bb7b0ef main -> main
>> ```
>>
>> Having the link in the console saves me tremendous time and is
>> extremely effective/efficient. Can we get links in the console plz?
>>
>
> If we look very carfully at the log, we see that all the messages prefixed
> with "remote:" come from the remote (git server).
> In your case github. Other "Git repo servers" like gitlab or bitbucket have
> the same feature.
>
> Git itself, running on your local computer, does not produce this links.
> If you really want to push directly to the main branch and want to see the
> a link, then you better talk to the gihub folks.
Thanks for a good suggestion.
Another piece of advice we may want to give is that a good bugreport
should illustrate both good and bad case. With only the above one,
I couldn't guess that the complaint was about lack of information
that comes over the wire from remote, *exactly* because the report
failed to show the good case that shows the information and made the
readers guess what thing that does not exist in the output is being
complained about. A good bugreport should not force readers guess.
If the report had both good and bad cases, it would have helped
readers help the reporter much easier.
Thanks.
next prev parent reply other threads:[~2023-10-30 6:56 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-10-29 23:15 please add link / url to remote - when - git push Alexander Mills
2023-10-30 6:36 ` Torsten Bögershausen
2023-10-30 6:55 ` Junio C Hamano [this message]
2023-10-30 8:52 ` Michal Suchánek
2023-10-30 9:06 ` Jeff King
2023-10-30 9:30 ` Michal Suchánek
2023-10-30 9:39 ` Jeff King
2023-10-30 9:45 ` Michal Suchánek
2023-10-30 9:54 ` Jeff King
2023-10-30 10:42 ` Michal Suchánek
2023-10-30 15:55 ` Taylor Blau
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=xmqqa5s0egzn.fsf@gitster.g \
--to=gitster@pobox.com \
--cc=alexander.d.mills@gmail.com \
--cc=git@vger.kernel.org \
--cc=tboegi@web.de \
/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).