Use consistent indentation in the file `HelpersCommon.psm1` by daxian-dbw · Pull Request #26608 · PowerShell/PowerShell
PowerShell / PowerShell Public
-
Notifications
You must be signed in to change notification settings - Fork 8.4k
Merged
Conversation
daxian-dbw
commented
Dec 12, 2025
daxian-dbw
commented
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
- PR has a meaningful title
- Use the present tense and imperative mood when describing your changes
- Summarized changes
- Make sure all
.h,.cpp,.cs,.ps1and.psm1files have the correct copyright header - This PR is ready to merge. If this PR is a work in progress, please open this as a Draft Pull Request and mark it as Ready to Review when it is ready to merge.
- Breaking changes
- None
- OR
- Experimental feature(s) needed
- Experimental feature name(s):
- User-facing changes
- Not Applicable
- OR
- Documentation needed
- Issue filed:
- Testing - New and feature
- N/A or can only be tested interactively
- OR
- Make sure you've added a new test if existing tests do not effectively test the code changed
daxian-dbw
requested review from
a team and
jshigetomi
as code owners
Copilot AI review requested due to automatic review settings
daxian-dbw
added
the
CL-CodeCleanup
label
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
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
reviewed
Copilot
AI
left a comment
Copilot AI left a comment
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
approved these changes
iSazonov
self-requested a review
iSazonov
approved these changes
iSazonov
commented
Dec 12, 2025
iSazonov commented
Collaborator
@daxian-dbw Ready to merge but blocked by code owner.
shane7777777777777
commented
Dec 12, 2025
shane7777777777777 commented
approve
shane7777777777777
approved these changes
shane7777777777777
commented
Dec 12, 2025
shane7777777777777 commented
appove
adityapatwardhan
approved these changes
daxian-dbw
deleted the
cleanup
branch
kilasuit pushed a commit to kilasuit/PowerShell that referenced this pull request
JustinGrote pushed a commit to JustinGrote/PowerShell that referenced this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment