◐ Shell
clean mode source ↗

Use consistent indentation in the file `HelpersCommon.psm1` by daxian-dbw · Pull Request #26608 · PowerShell/PowerShell

Skip to content

Navigation Menu

Sign in

Appearance settings

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up

Appearance settings

Conversation

@daxian-dbw

Copy link Copy Markdown

Member

PR Summary

The file HelpersCommon.psm1 use both 4 spaces and tab characters as indentations in the file, which makes it hard to view the file from an editor where a tab character is not the same width as 4 spaces. This PR fixes it to always use 4 spaces.

PR Checklist

@daxian-dbw daxian-dbw requested review from a team and jshigetomi as code owners

December 12, 2025 00:39

Copilot AI review requested due to automatic review settings

December 12, 2025 00:39

@daxian-dbw daxian-dbw added the CL-CodeCleanup

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

label

Dec 12, 2025

@daxian-dbw

daxian-dbw commented

Dec 12, 2025

edited by unfurl-links Bot

Loading

Copy link Copy Markdown

Member Author

@iSazonov can you please review? It's easy to view by ignoring the white space changes:
https://github.com/PowerShell/PowerShell/pull/26608/changes?w=1

GitHub

PR Summary The file HelpersCommon.psm1 use both 4 spaces and tab characters as indentations in the file, which makes it hard to view the file from an editor where a tab character is not the same wi...

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 wasn't able to review any files in this pull request.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@iSazonov iSazonov self-requested a review

December 12, 2025 04:13

@iSazonov

Copy link Copy Markdown

Collaborator

@daxian-dbw Ready to merge but blocked by code owner.

daxian-dbw reacted with thumbs up emoji

@shane7777777777777

Copy link Copy Markdown

approve

@shane7777777777777

Copy link Copy Markdown

appove

@daxian-dbw daxian-dbw merged commit 15d0a28 into PowerShell:master

Dec 12, 2025

42 of 43 checks passed

@daxian-dbw daxian-dbw deleted the cleanup branch

December 12, 2025 18:40

kilasuit pushed a commit to kilasuit/PowerShell that referenced this pull request

Jan 2, 2026

JustinGrote pushed a commit to JustinGrote/PowerShell that referenced this pull request

Jun 2, 2026

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

Reviewers

Copilot code review Copilot Copilot left review comments

@adityapatwardhan adityapatwardhan adityapatwardhan approved these changes

@iSazonov iSazonov iSazonov approved these changes

@jshigetomi jshigetomi Awaiting requested review from jshigetomi jshigetomi is a code owner

+1 more reviewer

@shane7777777777777 shane7777777777777 shane7777777777777 approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

CL-CodeCleanup

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

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@daxian-dbw @iSazonov @shane7777777777777 @adityapatwardhan