PowerShell / PowerShell/PSScriptAnalyzer
PSPlaceCloseBrace can't handle empty lines
未关闭
还没有人认领这个 Issue。
Area - Formatter
Issue - Bug
- 主要语言
- C#
- 星标
- 2.2k
- 派生
- 414
- 平均合并
- 13 小时 1 分钟
- 30 天内合并 PR
- 2
描述
Basically, it gets fixed if I have:
}
else {
But it doesn't get fixed if I have:
}
else {
And I would really like it to be able to handle:
}
# An explanation of the next code block
else {
And format it like:
} else {
# An explanation of the next code block
Steps to reproduce
Invoke-Formatter {
if (Test-Path ReadMe.md) {
Show-Markdown ReadMe.md
}
elseif (Test-Path ReadMe*) {
Get-Item ReadMe* | Select-Object -First 1 | Get-Content
}
} -Settings @{
Rules = @{
PSPlaceCloseBrace = @{
Enable = $true
NoEmptylineBefore = $true
NewLineAfter = $false
}
}
}
Expected behavior
It should output
if (Test-Path ReadMe.md) {
Show-Markdown ReadMe.md
} elseif (Test-Path ReadMe*) {
Get-Item ReadMe* | Select-Object -First 1 | Get-Content
}
Actual behavior
if (Test-Path ReadMe.md) {
Show-Markdown ReadMe.md
}
elseif (Test-Path ReadMe*) {
Get-Item ReadMe* | Select-Object -First 1 | Get-Content
}
Environment data
> $PSVersionTable
$PSVersionTable
Name Value
---- -----
PSVersion 7.1.0
PSEdition Core
GitCommitId 7.1.0
OS Microsoft Windows 10.0.19041
Platform Win32NT
PSCompatibleVersions {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion 2.3
SerializationVersion 1.1.0.1
WSManStackVersion 3.0
> (Get-Module -ListAvailable PSScriptAnalyzer).Version | ForEach-Object { $_.ToString() }
1.19.1
1.18.1
1.18.0
1.17.1
贡献指南
从这里开始
- 先读完整个 Issue,再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库,在一个分支上完成修改。
- 提交 Pull Request,并在描述里引用这个 Issue 编号。
调研方向
使用 Invoke-Formatter 和 issue 中所示的 PSPlaceCloseBrace 设置复现该行为,并比较空行和注释的情况。跟踪 PSScriptAnalyzer 针对该规则的格式化器路径;当空行或中间的注释不再阻止闭合大括号与 else 按预期连接时,即表示完成。
由索引模型根据 Issue 内容生成。
评估
- 技术栈
- csharp, powershell
- 领域
- tooling
- Issue 类型
- 缺陷
- 难度
- 3/5
- 预计耗时
- 1-2 天
- 活跃度
- 停滞
- 描述清晰度
- 基本清楚
- 新手友好度
- 48/100