Sponsored Content
Skip to content

chore: Ignore identifier-naming update in git blame - #7066

Merged
bthomee merged 2 commits into
XRPLF:developfrom
godexsoft:chore/clang-tidy-ignore-rev
May 3, 2026
Merged

chore: Ignore identifier-naming update in git blame#7066
bthomee merged 2 commits into
XRPLF:developfrom
godexsoft:chore/clang-tidy-ignore-rev

Conversation

@godexsoft

Copy link
Copy Markdown
Contributor

High Level Overview of Change

In #6571 we changed most of the files in the project (identifier naming). This PR adds that commit git-blame-ignore-revs such that it does not pollute git's history in git blame and preserves the original authors instead.

API Impact

No impact.

@godexsoft
godexsoft requested review from bthomee and Copilot May 3, 2026 16:43
@godexsoft godexsoft added the Trivial Simple change with minimal effect, or already tested. Only needs one approval. label May 3, 2026

Copilot AI left a comment

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.

Pull request overview

Adds the large-scale identifier renaming commit from #6571 to the repository’s .git-blame-ignore-revs so that git blame can ignore it and preserve original authorship signal.

Changes:

  • Add the #6571 commit SHA to .git-blame-ignore-revs with an explanatory comment.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .git-blame-ignore-revs Outdated

@bthomee bthomee left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Copilot's suggestion looks valid, so please apply it.

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings May 3, 2026 21:15

Copilot AI left a comment

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.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@bthomee bthomee added the Ready to merge *PR author* thinks it's ready to merge. Has passed code review. Perf sign-off may still be required. label May 3, 2026
@bthomee
bthomee added this pull request to the merge queue May 3, 2026
Merged via the queue into XRPLF:develop with commit 8490206 May 3, 2026
1 check passed
bthomee added a commit that referenced this pull request May 4, 2026
Co-authored-by: Bart <bthomee@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
beartec-jpg pushed a commit to beartec-jpg/FalconLedger that referenced this pull request Jun 1, 2026
Co-authored-by: Bart <bthomee@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Ready to merge *PR author* thinks it's ready to merge. Has passed code review. Perf sign-off may still be required. Trivial Simple change with minimal effect, or already tested. Only needs one approval.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants