Bug 39096 - Add "tidy all" commits to a git blame ignore file
Summary: Add "tidy all" commits to a git blame ignore file
Status: Needs Signoff
Alias: None
Product: Koha
Classification: Unclassified
Component: Architecture, internals, and plumbing (show other bugs)
Version: unspecified
Hardware: All All
: P5 - low enhancement
Assignee: Jonathan Druart
QA Contact: Testopia
URL:
Keywords:
Depends on: 38664
Blocks:
  Show dependency treegraph
 
Reported: 2025-02-11 15:23 UTC by Jonathan Druart
Modified: 2025-02-11 15:28 UTC (History)
1 user (show)

See Also:
Change sponsored?: ---
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:
Circulation function:


Attachments
Bug 39096: Add 'tidy all' commits to a git blame ignore file (1.36 KB, patch)
2025-02-11 15:28 UTC, Jonathan Druart
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Jonathan Druart 2025-02-11 15:23:45 UTC
To get a relevant blame we want to exclude some commits from bug 38664.
Comment 1 Jonathan Druart 2025-02-11 15:28:26 UTC
Created attachment 177739 [details] [review]
Bug 39096: Add 'tidy all' commits to a git blame ignore file

To get a relevant blame we want to exclude some commits from bug 38664.

Test plan:
0. Do not apply this patch
1.
% git blame C4/Auth.pm (or other file)
Notice that there is a lot of reference to bug 38664 (you can see it
with the author "Koha Development Team")
2. Apply this patch
3. Tell git to use this new file:
 % git config blame.ignoreRevsFile .git-blame-ignore-revs

Repeat 1.
=> There are way less refs to "Koha Development Team"

Before:
% git blame C4/Auth.pm|grep "Koha Development Team" | wc -l
454
After:
% git blame C4/Auth.pm|grep "Koha Development Team" | wc -l
29

QA: Do we add the `git config` command that to ktd so devs won't need to do it?