PowerShell / PowerShell/PSScriptAnalyzer

Update/standardize the internal custom `VariableAnalysis` classes

Đang mở
#1,283 0 bình luận 0 reaction 0 người được giao Xem trên GitHub

Chưa có ai nhận issue này.

Area - Engine Consider - 2.0 Issue - Enhancement
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ả

This addresses technical debt.

... the VariableAnalysis classes that were forked from PowerShell's own classes 4 years ago. Those classes were not updated since and have small bugs that leads to certain types of analysis not being possible. Unfortunately, the original maintainers have taken only a subset of it and added a lot of custom code to it. I once tried to just delete it and replace it with the code from PowerShell and some of the challenges were that SMA are very entangled with each other in terms of dependencies and it compiles against .Net Core, but PSSA compiles against .Net Standard.

(https://github.com/PowerShell/PSScriptAnalyzer/pull/1277#issuecomment-508929230)

Hướng dẫn đóng góp

Mở hướng dẫn đóng góp

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. 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.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. 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 đọc cuộc thảo luận được liên kết trong PowerShell/PSScriptAnalyzer#1277 và xác định các lớp VariableAnalysis nội bộ cùng các phần phụ thuộc của chúng. So sánh các lớp được fork với các phiên bản của PowerShell, đồng thời tính đến các mục tiêu biên dịch .NET Core và .NET Standard. Công việc được coi là hoàn tất khi các khoản nợ kỹ thuật, các lỗi phân tích đã biết và các ràng buộc về phần phụ thuộc hoặc framework được xử lý mà không làm hỏng analyzer.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Đánh giá

Công nghệ
csharp, powershell
Lĩnh vực
tooling
Loại issue
Tái cấu trúc
Độ khó
5/5
Thời gian dự kiến
Hơn một tuần
Mức độ hoạt động
Đình trệ
Độ rõ ràng
Cần làm rõ
Mức phù hợp với người mới
20/100

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.