PowerShell / PowerShell/PSScriptAnalyzer
PSPlaceCloseBrace can't handle empty lines
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- C#
- Sterne
- 2.2k
- Forks
- 414
- Ø Merge
- 13 Std. 1 Min.
- Gemergte PRs (30 T.)
- 2
Beschreibung
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
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Reproduziere das Verhalten mit Invoke-Formatter und den im Issue gezeigten PSPlaceCloseBrace-Einstellungen und vergleiche Fälle mit Leerzeilen und Kommentaren. Verfolge den Formatierungspfad von PSScriptAnalyzer für diese Regel; abgeschlossen ist die Aufgabe, wenn eine Leerzeile oder ein dazwischenstehender Kommentar nicht mehr verhindert, dass die schließende geschweifte Klammer und else wie erwartet zusammengeführt werden.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- csharp, powershell
- Bereich
- tooling
- Issue-Typ
- Bug
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 48/100