Skip to content

Conversation

@maxisbey
Copy link
Contributor

Adds commit 89e9c43 from PR #1553 to .git-blame-ignore-revs to exclude the large-scale addition of # pragma: no cover comments from git blame output.

Motivation and Context

PR #1553 established a 100% code coverage baseline by adding pragma comments throughout the codebase. While these annotations are valuable for coverage tracking, they add noise to git blame output by attributing lines to a formatting commit rather than showing the original authors of the substantive code.

Adding this commit to .git-blame-ignore-revs ensures that git blame (when configured to use the file) will skip over these pragma additions and show the actual authors of the underlying code logic.

How Has This Been Tested?

This is a documentation/tooling change that affects git blame output only. The change can be verified by:

  • Running git blame --ignore-revs-file .git-blame-ignore-revs <file> on files modified in 89e9c43
  • Configuring git globally: git config blame.ignoreRevsFile .git-blame-ignore-revs

Breaking Changes

None.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update

Checklist

  • I have read the MCP Documentation
  • My code follows the repository's style guidelines
  • New and existing tests pass locally
  • I have added appropriate error handling
  • I have added or updated documentation as needed

Additional context

This follows the same pattern as the existing entry for commit 5439619 (120 line-length formatting). Developers can configure git to automatically use this file with:

git config blame.ignoreRevsFile .git-blame-ignore-revs

Adds commit 89e9c43 to
.git-blame-ignore-revs to exclude the large-scale addition of
# pragma: no cover comments from git blame output.

This commit established a 100% code coverage baseline by adding pragma
comments throughout the codebase. While important for coverage tracking,
these annotations are noise in git blame and should be ignored to show
the original authors of substantive code changes.

Github-Issue: #1553
@maxisbey maxisbey enabled auto-merge (squash) November 11, 2025 13:36
@maxisbey maxisbey merged commit 720d713 into main Nov 11, 2025
38 of 39 checks passed
@maxisbey maxisbey deleted the git-blame-ignore-coverage-baseline branch November 11, 2025 14:03
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