Skip to content

CSHARP-5610: Cleanup unnecessary whitespaces in the codebase #1707

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 1 commit into from
Jun 24, 2025

Conversation

adelinowona
Copy link
Contributor

This PR accompanies: #1697 as the first change to be ignored in git blame.

@adelinowona adelinowona requested a review from a team as a code owner June 9, 2025 21:14
@adelinowona adelinowona requested review from sanych-sun and removed request for a team June 9, 2025 21:14
@adelinowona adelinowona added the chore Label to hide PR from generated Release Notes label Jun 9, 2025
@adelinowona adelinowona requested review from BorisDog and removed request for sanych-sun June 9, 2025 21:14
@adelinowona adelinowona changed the title Cleanup unnecessary whitespaces in the codebase CSHARP-5610: Cleanup unnecessary whitespaces in the codebase Jun 9, 2025

Copy link
Contributor Author

@adelinowona adelinowona Jun 9, 2025

Choose a reason for hiding this comment

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

git blame has difficulties blaming this type of space changes. Git fundamentally sees this as creating a new empty line without spaces and deleting an old line with spaces rather than simply modifying the existing line. As such, git blame will fail to ignore this change.

Copy link
Contributor

@BorisDog BorisDog left a comment

Choose a reason for hiding this comment

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

LGTM

@adelinowona adelinowona merged commit 1186b93 into mongodb:main Jun 24, 2025
1 of 2 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
chore Label to hide PR from generated Release Notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants