Skip to content

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 akx force-pushed the git-blame-ignore-revs branch from 5ee00bb to a331d45 Compare November 8, 2023 12:00
@akx
Copy link
Contributor Author

akx commented Nov 8, 2023

@homm Great, thanks! Added those too.

@akx akx force-pushed the git-blame-ignore-revs branch from 3680097 to 75f5e17 Compare November 13, 2023 15:22
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"

Co-authored-by: Andrew Murray <[email protected]>
@radarhere radarhere merged commit b8826ef into python-pillow:main Dec 24, 2023
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.

3 participants