Skip to content

Add commit to use SPDX copyright headers to git ignore blame revs.#7381

Merged
rapids-bot[bot] merged 1 commit intorapidsai:mainfrom
csadorf:chore/add-spdx-copyright-header-fix-commit-to-git-blame-ignore-revs
Oct 24, 2025
Merged

Add commit to use SPDX copyright headers to git ignore blame revs.#7381
rapids-bot[bot] merged 1 commit intorapidsai:mainfrom
csadorf:chore/add-spdx-copyright-header-fix-commit-to-git-blame-ignore-revs

Conversation

@csadorf
Copy link
Copy Markdown
Contributor

@csadorf csadorf commented Oct 23, 2025

No description provided.

@csadorf csadorf requested a review from jcrist October 23, 2025 21:22
@csadorf csadorf added improvement Improvement / enhancement to an existing function non-breaking Non-breaking change labels Oct 23, 2025
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Oct 23, 2025

Important

Review skipped

Auto reviews are disabled on this repository.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown

@greptile-apps greptile-apps Bot left a comment

Choose a reason for hiding this comment

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

Greptile Overview

Greptile Summary

This PR adds a single commit SHA (6f7a1052f545c7d51d6893ac5791ace574671ed5) to the .git-blame-ignore-revs file to exclude a bulk SPDX copyright header conversion from git blame history. The .git-blame-ignore-revs file is a Git configuration mechanism that tells git blame to skip specific commits when attributing line authorship, preventing mechanical formatting changes from obscuring the actual code authors. This change integrates with the existing pattern in the file (comment + SHA) and follows a common practice in large codebases where sweeping stylistic refactors would otherwise pollute blame history. The commit being ignored touched many files purely to update copyright headers to SPDX format, which is a legal/licensing convention change rather than a functional code change.

Important Files Changed

Filename Score Overview
.git-blame-ignore-revs 5/5 Added commit SHA for SPDX copyright header conversion to Git blame ignore list

Confidence score: 5/5

  • This PR is safe to merge with zero risk—it only affects developer tooling (git blame) and has no impact on runtime behavior or build outputs.
  • Score reflects that this is a standard maintenance operation with an established pattern, clear documentation, and no possibility of breaking changes.
  • No files require special attention—the change is trivial and follows existing conventions.

Sequence Diagram

sequenceDiagram
    participant User
    participant Git
    participant BlameIgnoreRevs as ".git-blame-ignore-revs"
    
    User->>BlameIgnoreRevs: "Add commit hash for SPDX copyright headers"
    Note over BlameIgnoreRevs: Append commit 6f7a1052f545c7d51d6893ac5791ace574671ed5
    User->>Git: "git blame --ignore-revs-file .git-blame-ignore-revs"
    Git->>BlameIgnoreRevs: "Read ignored commit hashes"
    BlameIgnoreRevs-->>Git: "Return list of commits to ignore"
    Git-->>User: "Show blame output excluding SPDX header changes"
Loading

1 file reviewed, no comments

Edit Code Review Agent Settings | Greptile

@csadorf
Copy link
Copy Markdown
Contributor Author

csadorf commented Oct 23, 2025

/merge

@rapids-bot rapids-bot Bot merged commit 3f03cdb into rapidsai:main Oct 24, 2025
294 of 302 checks passed
@csadorf csadorf deleted the chore/add-spdx-copyright-header-fix-commit-to-git-blame-ignore-revs branch October 24, 2025 17:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

improvement Improvement / enhancement to an existing function non-breaking Non-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants