PowerShell / PowerShell/PSScriptAnalyzer
Whitespace Between Parameters Is Trimming Required Quotes
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
- ✅ Make sure you are able to repro it on the latest released version
- ✅ Perform a quick search for existing issues to check if this bug has already been reported.
- Might be related to #1540
This was one of the more simple examples I could find.
I found this issue popping up in random files and finally realized it was a formatter issue.
Steps to reproduce
Describe "DescribeName" {
Context "ContextName" {
It "ItName" {
Assertion
}
It "output: attributes matches $($outputs.attributes)" {
$results.attributes | Should -Be $outputs.attributes
}
}
}
Expected behavior
Describe "DescribeName" {
Context "ContextName" {
It "ItName" {
Assertion
}
It "output: attributes matches $($outputs.attributes)" {
$results.attributes | Should -Be $outputs.attributes
}
}
}
Actual behavior
Describe "DescribeName" {
Context "ContextName" {
It "ItName" {
Assertion
}
It "output: attributes matches $($outputs.attributes){ <----- it removed valid quote, resulting in breaking syntax
$results.attributes | Should -Be $outputs.attributes
}
}
}
Environment data
> $PSVersionTable
Name Value
---- -----
PSVersion 7.0.3
PSEdition Core
GitCommitId 7.0.3
OS Darwin 19.5.0 Darwin Kernel Version 19.5.0: Tue May 26 20:41:44 PDT 2020; root:xnu-6153.121.2~2/RELEASE_X86_64
Platform Unix
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.0
1.18.3
1.19.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
Beginne damit, das Formatter-Problem mit dem PowerShell-Beispiel aus dem Issue unter Verwendung der aufgeführten PSScriptAnalyzer-Versionen zu reproduzieren. Verfolge die Formatierung der Zeichenkette It in Anführungszeichen und füge einen Regressionstest für den erforderlichen Whitespace und das Anführungszeichen hinzu. Erledigt bedeutet, dass die Formatierung das schließende Anführungszeichen beibehält und das Ergebnis weiterhin gültige PowerShell-Syntax ist.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- powershell
- Bereich
- tooling
- Issue-Typ
- Bug
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Klar beschrieben
- Anfängerfreundlichkeit
- 45/100