All of lore.kernel.org
 help / color / mirror / Atom feed
From: <rsbecker@nexbridge.com>
To: "'Shafiq Jetha'" <shafiq.jetha@benevity.com>, <git@vger.kernel.org>
Subject: RE: Suggestion to provide a way for screenshots to be added to commit messages
Date: Fri, 5 May 2023 11:23:27 -0400	[thread overview]
Message-ID: <001a01d97f65$96b77450$c4265cf0$@nexbridge.com> (raw)
In-Reply-To: <MWHPR01MB2318693B28B740F75D90E1DCF3729@MWHPR01MB2318.prod.exchangelabs.com>

On Friday, May 5, 2023 10:51 AM, Shafiq Jetha wrote:
>It would be nice if we could add images directly to commit messages via
git. The
>reason for this is so that we can view the history and see real examples of
important
>UI elements. Usually during a PR we provide screenshots so that people can
see how
>the change affects UI elements, or to show the output of specific commands
that
>might show specific data in specific scenarios. I can also see this being
useful for
>people that work on projects where code produces some sort of visual output
where
>words cannot accurately describe the changes (a picture is worth 1000
words, right?).
>
>It would be great if this sort of an integration could be considered, and I
know that
>this would mean updating the clients, specifically the command-based
clients, to
>interpret these images and provide a way to render them or display a link
to them in
>some fashion. There's also the question of whether to embed the images
directly in
>the commit message or to link out to them and download them as artefacts of
the
>repo. And then there is the everlasting question of whether or not a git
repo should
>contain build artefacts at all (since it can be argued that an image is a
build
>artefact).
>Lots of things to consider but I hope that I have at least started a
dialogue for this to
>be explored further.

My team has a similar use case. We adopted a few ticket systems - depending
on our git upstream - including Jira and GitHub Issues. Both provide the
capability to upload images and have discussions during the PR referencing
the original ticket via the commit content. The commit comment simply needs
to reference the issue through the integration provided by the ticket
systems. Something like this might solve your need and is off the shelf.

--Randall


  reply	other threads:[~2023-05-05 15:26 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-05-05 14:51 Suggestion to provide a way for screenshots to be added to commit messages Shafiq Jetha
2023-05-05 15:23 ` rsbecker [this message]
2023-05-06  9:01 ` Erik Cervin Edin
2023-05-06 11:54   ` Teng Long
     [not found]     ` <xmqqpm7drvz7.fsf@gitster.g>
2023-05-08 14:19       ` Shafiq Jetha

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='001a01d97f65$96b77450$c4265cf0$@nexbridge.com' \
    --to=rsbecker@nexbridge.com \
    --cc=git@vger.kernel.org \
    --cc=shafiq.jetha@benevity.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.