1508324 - Generate a list of past changesets to be skipped for blame

Status: RESOLVED FIXED

(Developer Services :: General, task)

Description

[:Felipe Gomes (needinfo for replies!)]

Bug 1508002 is creating an hg alias to use the new `hg annotate --skip` support with a pre-established convention of skipping changesets which have "skip-blame" in the commit summary, or changesets listed in a file in the tree.

This file will be used to provide extra changesets that should also be skipped but did not include "skip-blame" in the summary.


CC'ing some people who I know have made large code-formatting changes in the past to help gather this list.

Comment 1

[Florian Quèze [:florian]]

Attachment: Scripted changes

This file includes a list of all the large changesets I produced using scripts, and at the end of the file I also added a couple large changesets that I'm aware of even though I'm not the author.

If this is going to be used only for the annotate command and not for the log command, we may want to reduce the list to include only changesets that caused changes on lines that still exist (eg. removing Cc and Ci declarations doesn't need to be skipped in annotate, but it's noise when looking at the log).

Comment 2

[Andrew McCreight [:mccr8] (PTO until 12-2)]

I feel like there could be some kind of heuristic like patches that touch 30+ files, and then scrape the commit history to get a list of potential candidates.

Comment 3

[:Felipe Gomes (needinfo for replies!)]

Attachment: .hg-annotate-ignore-revs

Before populating this file with a lot more items, let's get the file created and filled with the same changeset currently in .git-blame-ignore-revs, so that newer additions know what's the expected format of this file.

Unfortunately this file can't have full-line comments like the .git-blame-ignore-revs because the extdata() feature from mercurial won't accept any invalid chars at the beginning of the file

Comment 4

[(no longer active)]

Comment on attachment 9028835 [details] [diff] [review]
.hg-annotate-ignore-revs

Review of attachment 9028835 [details] [diff] [review]:
-----------------------------------------------------------------

This isn't too bad, it's self-commenting!

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/6baf9f33a6cd
Create the .hg-annotate-ignore-revs file to be used with hg smart-annotate. r=ehsan DONTBUILD NPOTB

Comment 6

[Noemi Erli[:noemi_erli]]

https://hg.mozilla.org/mozilla-central/rev/6baf9f33a6cd

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/4fac48f96e20
Generate list of past changesets to be skipped for blame. r=me NPOTB DONTBUILD

Comment 8

[Eliza Balazs [:ebalazs_]]

https://hg.mozilla.org/mozilla-central/rev/4fac48f96e20