tools: add .git-blame-ignore-revs #5154
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
See
man git-blame,git blame --ignore-revs-file <file>and theblame.ignoreRevsFilegit config option.Step 3 of 3+x, as detailed in #5145
These are currently the commits with the biggest number of changed files:
I had several different versions of the ignore list, but in the end I decided to remove some commit IDs again where non-syntax-related stuff was included or where the changes didn't affect the blame history much, like commits with many added new lines, or changes in terms of relevance, like import statements, etc.
I'm pretty confident with the current list of commit IDs now. Having a blame ignore list can already be confusing when checking a file's (blame-)history, so the ignored bulk-changes should all be trivial.
Example:
https://github.com/bastimeyer/streamlink/blame/git-blame-ignore-revs/src/streamlink/plugins/adultswim.py#L46
~), same as current master:https://github.com/bastimeyer/streamlink/blame/git-blame-ignore-revs~/src/streamlink/plugins/adultswim.py#L46