Skip to content

Remove whitespace and git merge markers in src/runtime files - #8917

Merged
akoeplinger merged 3 commits into
mainfrom
akoeplinger-patch-1
Sep 1, 2026
Merged

akoeplinger merged 3 commits into
mainfrom
akoeplinger-patch-1

Conversation

@akoeplinger

Copy link
Copy Markdown
Member

It looks like these were caused by conflict resolution and show up in VMR backflows now: dotnet/runtime#132978

Copilot AI lite review requested due to automatic review settings September 1, 2026 13:59
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 1 pipeline(s).
1 pipeline(s) were filtered out due to trigger conditions.
There may be pipelines that require an authorized user to comment /azp run to run.

@akoeplinger akoeplinger changed the title Remove whitespace in src/runtime files Remove whitespace and git merge markers in src/runtime files Sep 1, 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.

🟢 Approval recommended

Pull request overview

Removes stray whitespace-only lines in src/runtime CoreCLR sources that were likely introduced during conflict resolution and are now surfacing via VMR backflows (per linked runtime PR).

Changes:

  • Removed a whitespace-only line in CallDefaultConstructor in callhelpers.cpp.
  • Normalized a whitespace-only blank line in the clrconfig.cpp helper block.
File summaries
File Description
src/runtime/src/coreclr/vm/callhelpers.cpp Removes an extraneous whitespace-only line inside CallDefaultConstructor.
src/runtime/src/coreclr/utilcode/clrconfig.cpp Replaces a whitespace-only line with a clean blank line to normalize formatting.
Review details
  • Files reviewed: 2/3 changed files
  • Comments generated: 0
  • Review effort level: Lite

💡 Add a code-review agent skill for context-aware, tailored reviews. Learn more in the docs.

@akoeplinger
akoeplinger enabled auto-merge (squash) September 1, 2026 18:32
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