Exclude major black only commits from git blame #2780
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
As more projects I've used have adopted `black` for styling, one minor annoyance has been the impact that `black` commits that impact many lines of code have on the usability of `git blame`. A co-worker introduced me to the changes proposed here, which make the blame more usable. As of spring last year, github now looks at .git-blame-ignore-devs by default (https://docs.github.com/en/repositories/working-with-files/using-files/viewing-a-file#ignore-commits-in-the-blame-view), so the blame view in github will use these commits. devs who want their local `git blame` will need the step added to the CONTRIBUTING.md file.
martinfleis
approved these changes
Feb 5, 2023
|
Thanks also for sharing, I knew this was a problem, didn't know there was a nice solution |
m-richards
reviewed
Feb 6, 2023
m-richards
approved these changes
Feb 6, 2023
Co-authored-by: Matt Richards <45483497+m-richards@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
As more projects I've used have adopted
blackfor styling, one minorannoyance has been the impact that
blackcommits that impact manylines of code have on the usability of
git blame. A co-workerintroduced me to the changes proposed here, which make the blame more
usable. As of spring last year, github now looks at
.git-blame-ignore-devs by default
(https://docs.github.com/en/repositories/working-with-files/using-files/viewing-a-file#ignore-commits-in-the-blame-view),
so the blame view in github will use these commits. devs who want their
local
git blamewill need the step added to the CONTRIBUTING.md file.