PowerShell / PowerShell/PSScriptAnalyzer
unary not is fooling tracker of variable use when assignment is in foreach loop
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
Steps to reproduce
Get-AzAutomationSchedule $resourceGroup $automationAccount | ForEach-Object {
$PrimaryExists = $true
}
if (-not $PrimaryExists) {
Write-Output "hello"
}
Expected behavior
$PrimaryExists should not be flagged as assigned but never used.
Actual behavior
It is flagged.
Environment data
> $PSVersionTable
Name Value
---- -----
PSVersion 6.2.3
PSEdition Core
GitCommitId 6.2.3
OS Linux 4.15.0-1056-oem #65-Ubuntu SMP Thu Sep 12…
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.18.3
Notes
I tried to create a more concise example, but the assignment needs to be a foreach { } at the end of a pipeline to exhibit the error.
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
Beginnen Sie damit, die bereitgestellte PowerShell-Pipeline auszuführen und die Analyzer-Diagnose für die Zuweisung innerhalb des abschließenden ForEach-Object-Blocks zu prüfen. Verfolgen Sie die Variablenverwendungsanalyse, die die Warnung assigned-but-unused erzeugt, fügen Sie dann einen Regressionstest für dieses Beispiel hinzu und bestätigen Sie, dass die Warnung nicht mehr gemeldet wird, während gewöhnliche ungenutzte Zuweisungen weiterhin als Fehler markiert werden.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- powershell
- Bereich
- tooling
- Issue-Typ
- Bug
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 35/100