All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jonathan Nieder <jrnieder@gmail.com>
To: Enji Cooper <yaneurabeya@gmail.com>
Cc: git@vger.kernel.org, Barret Rhoden <brho@google.com>
Subject: Re: blame --ignore-rev - repo-level ignore file
Date: Fri, 17 Apr 2020 17:52:53 -0700	[thread overview]
Message-ID: <20200418005253.GB121175@google.com> (raw)
In-Reply-To: <388B3B98-4929-4B4A-9167-3076CFDD0B5B@gmail.com>

(+Barret)
Hi!

Enji Cooper wrote:

> Hi,

> Given the increased number of C/C++ and python projects that both
> use git and a tool like black (python formatting tool), or
> clang-format (C/C++ tool, primarily), I would like to seriously
> consider revisiting the topic "blame --ignore-rev - repo-level
> ignore file” topic that was originally started by Terry Davis here
> [1] (quoted above for historical reference with the reply provided
> by Barret Rhoden).
>
> Having a default configuration that can be used with git instead of
> forcing end-users to set custom configs per repo clone would make it
> considerably easier for end-users: one could clone a repo, reformat
> code with an automated tool in a series of known commits, add
> ignored revisions to the file when using git blame, then make
> additional changes building off the automated reformatters' changes.
>
> I will be more than happy to submit patches to make this possible; I
> just need some guidance in terms of what’s required when
> contributing changes, etc.

The first question would be what you expect the end-to-end workflow
to look like for users.  What commands would they run?  How would
they discover them?

As for the general contribution process:
https://www.kernel.org/pub/software/scm/git/docs/MyFirstContribution.html
is a good starting point.

Thanks and hope that helps,
Jonathan

> 1. https://marc.info/?l=git&m=156864623817929&q=mbox

      reply	other threads:[~2020-04-18  0:52 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <B2997F0D-AEEE-4431-85B8-98831ACF85B6@gmail.com>
2020-04-17 23:39 ` blame --ignore-rev - repo-level ignore file Enji Cooper
2020-04-18  0:52   ` Jonathan Nieder [this message]

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=20200418005253.GB121175@google.com \
    --to=jrnieder@gmail.com \
    --cc=brho@google.com \
    --cc=git@vger.kernel.org \
    --cc=yaneurabeya@gmail.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.