Skip to content

Remove the redundant #if UNIX check from the Unix only code - #28120

Open
Dongbo Wang (daxian-dbw) wants to merge 1 commit into
PowerShell:masterfrom
daxian-dbw:unix-ifdef-cleanup
Open

Dongbo Wang (daxian-dbw) wants to merge 1 commit into
PowerShell:masterfrom
daxian-dbw:unix-ifdef-cleanup

Conversation

@daxian-dbw

Copy link
Copy Markdown
Member

PR Summary

Remove the redundant #if UNIX check from the Unix only code.

Copilot AI lite review requested due to automatic review settings September 28, 2026 19:12
@daxian-dbw
Dongbo Wang (daxian-dbw) requested a review from a team as a code owner September 28, 2026 19:12
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
There may be pipelines that require an authorized user to comment /azp run to run.

@daxian-dbw Dongbo Wang (daxian-dbw) added the CL-CodeCleanup Indicates that a PR should be marked as a Code Cleanup change in the Change Log label Sep 28, 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.

Copilot review overview

🟢 Approval recommended

The reviewed change is narrowly scoped and has no unresolved issues.

Review effort: Lite
Findings: None

What changed in this PR

Removes a redundant Unix preprocessor guard from the Unix-only implementation.

Changes:

  • Simplifies Unix-only cursor repositioning logic.
File Description
src/​Microsoft.PowerShell.ConsoleHost/​host/​msh/​ConsoleHostRawUserInterface.cs Removes the redundant #if UNIX guard.

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CL-CodeCleanup Indicates that a PR should be marked as a Code Cleanup change in the Change Log

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants