Skip to content

Add ignore revs file - #1126

Merged
litvaOo merged 2 commits into
developfrom
dev/amd/add-ignore-revs
Oct 15, 2025
Merged

litvaOo merged 2 commits into
developfrom
dev/amd/add-ignore-revs

Conversation

@litvaOo

@litvaOo litvaOo commented Sep 25, 2025

Copy link
Copy Markdown
Contributor

Motivation

We had two major commits, impacting codestyle changes in every C++ file. Those changes create unnecessary noise in the git blame output, so in order to avoid that we want to exclude those commits from git blame.

Technical Details

Adds .git-ignore-revs file and explanation on how to use it in the CONTRIBUTING.md docs

Test Plan

No code changes

Test Result

No code changes

Submission Checklist

@litvaOo
litvaOo requested a review from a team as a code owner September 25, 2025 09:20
Copilot AI review requested due to automatic review settings September 25, 2025 09:20
@github-actions github-actions Bot added the documentation Improvements or additions to documentation label Sep 25, 2025

This comment was marked as off-topic.

Comment thread .git-blame-ignore-revs Resolved
Comment thread CONTRIBUTING.md Resolved
Comment thread .git-blame-ignore-revs
@@ -0,0 +1,2 @@
2ff2316227dc3bb04447060c719e78baf2d7383f
f7338717aeb56fbe394c0683539a0cf2f7f3ded7

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Please could you add 13c5e7a as well?
It is also a big formatting change

@litvaOo
litvaOo merged commit 59a30bb into develop Oct 15, 2025
3 checks passed
@litvaOo
litvaOo deleted the dev/amd/add-ignore-revs branch October 15, 2025 11:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation organization: ROCm

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants