Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

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

WIP: Fix: PSUseConsistentWhitespace formatting for do-while, unary operators, and native commands#2114

New issue

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

By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Open
DrSkillIssue wants to merge5 commits intoPowerShell:main
base:main
Choose a base branch
Loading
fromDrSkillIssue:format-miss-keywords

Conversation

@DrSkillIssue
Copy link

@DrSkillIssueDrSkillIssue commentedJun 17, 2025
edited
Loading

PR Summary

Fixes multiple whitespace formatting issues inPSUseConsistentWhitespace rule

Issues Fixed:

Changes Made:

Core Fixes:

  • AddedFindKeywordAfterBraceViolations to handle keywords following closing braces
  • Enhanced unary operator detection and spacing logic inFindOperatorViolations
  • FixedFindParameterViolations to skip whitespace inside string literals
  • Added comma pattern detection inFindSeparatorViolations for native command arguments
  • Added the additional keywords to openParenKeywordAllowList:Until, Do, Else, Catch, Finally

Tests Added:

  • Added test-cases for the above-mentioned changes.

PR Checklist

SamErde reacted with thumbs up emoji
@DrSkillIssueDrSkillIssue changed the titleFix: #2095 - UseConsistentWhitespace formatting for do-while/do-until loops and unary operatorsWIP: Fix: #2095 - UseConsistentWhitespace formatting for do-while/do-until loops and unary operatorsJun 17, 2025
@DrSkillIssue
Copy link
Author

DrSkillIssue commentedJun 17, 2025
edited
Loading

These PRs may just need closed as I couldn't reproduce the issues

PRs that were fixed with this PR

PRs that can be fixed in another PR(s) once this one is merged

Opting for multiple PRs as this one is large already.

@DrSkillIssueDrSkillIssue changed the titleWIP: Fix: #2095 - UseConsistentWhitespace formatting for do-while/do-until loops and unary operatorsWIP: Fix: PSUseConsistentWhitespace formatting for do-while, unary operators, and native commandsJun 17, 2025
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@andyleejordanandyleejordanAwaiting requested review from andyleejordan

@bergmeisterbergmeisterAwaiting requested review from bergmeisterbergmeister is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@DrSkillIssue

[8]ページ先頭

©2009-2025 Movatter.jp