Skip to content

Add git-blame-ignore-revs file to ignore the reformatting commit in git blame#4466

Merged
jlerbsc merged 1 commit intojavaparser:masterfrom
johannescoetzee:johannes/git-blame-ignore
Jun 12, 2024
Merged

Add git-blame-ignore-revs file to ignore the reformatting commit in git blame#4466
jlerbsc merged 1 commit intojavaparser:masterfrom
johannescoetzee:johannes/git-blame-ignore

Conversation

@johannescoetzee
Copy link
Copy Markdown
Collaborator

@johannescoetzee johannescoetzee commented Jun 12, 2024

The motivation is discussed in #4408

With this file in place, github will ignore 5106428 for determining where a change was made. For git cli, git config blame.ignoreRevFile needs to be used to configure this.

An example of this working can be found at https://github.com/johannescoetzee/javaparser/blame/johannes/git-blame-ignore/javaparser-core/src/main/java/com/github/javaparser/CommentsInserter.java (just an arbitrary file I chose on the relevant branch on my fork).

@jlerbsc jlerbsc merged commit 9868bde into javaparser:master Jun 12, 2024
@jlerbsc jlerbsc added this to the next release milestone Jun 12, 2024
@jlerbsc jlerbsc added PR: Added A PR that introduces new behaviour (e.g. functionality, tests) PR: Development A PR that affects only JavaParser developers (e.g. build / deploy infrastructure) and removed PR: Added A PR that introduces new behaviour (e.g. functionality, tests) labels Jun 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

PR: Development A PR that affects only JavaParser developers (e.g. build / deploy infrastructure)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants