PowerShell / PowerShell/PSScriptAnalyzer
unary not is fooling tracker of variable use when assignment is in foreach loop
オープン
まだ誰も着手していません。
Area - Scoping
Issue - Bug
- 主要言語
- C#
- スター
- 2.2k
- フォーク
- 414
- 平均マージ
- 13時間 1分
- マージ済み PR(30日)
- 2
説明
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.
コントリビューションガイド
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
調査の方向性
まず、提供されている PowerShell パイプラインを実行し、末尾の ForEach-Object ブロック内の代入に対するアナライザー診断を確認します。assigned-but-unused 警告を生成している変数使用分析を追跡し、次にこの例の回帰テストを追加して、通常の未使用の代入は引き続き検出されたまま、警告が報告されなくなったことを確認します。
索引モデルが issue の本文から書いたものです。
評価
- 技術スタック
- powershell
- 領域
- tooling
- issue の種類
- バグ
- 難易度
- 4/5
- 見積もり時間
- 3〜5日
- 活発さ
- 停滞
- 明瞭さ
- おおむね明確
- 初心者へのやさしさ
- 35/100