PowerShell / PowerShell/PSScriptAnalyzer
unary not is fooling tracker of variable use when assignment is in foreach loop
Chưa có ai nhận issue này.
- Ngôn ngữ chính
- C#
- Star
- 2.2k
- Fork
- 414
- Merge trung bình
- 13 giờ 1 phút
- Pull request đã merge (30 ngày)
- 2
Mô tả
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.
Hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Hướng nghiên cứu
Bắt đầu bằng cách chạy pipeline PowerShell được cung cấp và kiểm tra chẩn đoán của analyzer đối với phép gán bên trong khối ForEach-Object ở cuối. Theo dõi quá trình phân tích việc sử dụng biến tạo ra cảnh báo assigned-but-unused, sau đó thêm một regression test cho ví dụ này và xác nhận rằng cảnh báo không còn được báo cáo, trong khi các phép gán không được sử dụng thông thường vẫn tiếp tục bị đánh dấu.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Đánh giá
- Công nghệ
- powershell
- Lĩnh vực
- tooling
- Loại issue
- Lỗi
- Độ khó
- 4/5
- Thời gian dự kiến
- 3-5 ngày
- Mức độ hoạt động
- Đình trệ
- Độ rõ ràng
- Khá rõ ràng
- Mức phù hợp với người mới
- 35/100