Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add .git-blame-ignore-revs file #7528

Merged
merged 5 commits into from Dec 24, 2023

Conversation

akx
Copy link
Contributor

@akx akx commented Nov 6, 2023

This PR adds a .git-blame-ignore-revs file (the name is conventionally supported by e.g. GitHub).

The file is populated with the mechanical reformatting commit 53a7e35.

This makes blame tools (when configured to do so, e.g. git config blame.ignoreRevsFile .git-blame-ignore-revs) ignore the revision, making archaeology a bit easier.

@homm
Copy link
Member

homm commented Nov 8, 2023

There was also C code formatting commits #4770

@akx
Copy link
Contributor Author

akx commented Nov 8, 2023

@homm Great, thanks! Added those too.

.git-blame-ignore-revs Outdated Show resolved Hide resolved
Comment on lines 1 to 3
# Reformat with Black
53a7e3500437a9fd5826bc04758f7116bd7e52dc
# ClangFormat the C code
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
# Reformat with Black
53a7e3500437a9fd5826bc04758f7116bd7e52dc
# ClangFormat the C code
# Format with Black
53a7e3500437a9fd5826bc04758f7116bd7e52dc
# Format the C code with ClangFormat

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I just borrowed the commit messages, but sure :)

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

...my intention was to become closer to the commit messages. 53a7e35 is "Format with Black" and 46b7e86 is "Format with ClangFormat"

akx and others added 3 commits November 15, 2023 14:37
Co-authored-by: Andrew Murray <3112309+radarhere@users.noreply.github.com>
.git-blame-ignore-revs Outdated Show resolved Hide resolved
Co-authored-by: Andrew Murray <3112309+radarhere@users.noreply.github.com>
@radarhere radarhere merged commit b8826ef into python-pillow:main Dec 24, 2023
56 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants