PowerShell / PowerShell/PSScriptAnalyzer
feature request for formatting rule to have an empty line after brace
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ả
Summary of the new feature
This is a proposal to add additional formatting rule to have an empty line after brace to keep code more readable with OTBS scheme. It's a separate rule comparing to inserting a new line.
Current behaviour:
...
foreach ($account in $Permissions.Keys) {
foreach ($permission in $Permissions[$account].Keys) {
foreach ($folder in $Permissions[$account][$permission]) {
cNtfsPermissionEntry $folder {
Ensure = 'Present'
Path = $folder
Principal = $account
AccessControlInformation = @(
cNtfsAccessControlInformation {
FileSystemRights = $permission
}
)
}
}
}
}
...
Proposed behaviour:
...
foreach ($account in $Permissions.Keys) {
foreach ($permission in $Permissions[$account].Keys) {
foreach ($folder in $Permissions[$account][$permission]) {
cNtfsPermissionEntry $folder {
Ensure = 'Present'
Path = $folder
Principal = $account
AccessControlInformation = @(
cNtfsAccessControlInformation {
FileSystemRights = $permission
}
)
}
}
}
}
...
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 xác định các quy tắc định dạng và logic xử lý dấu ngoặc nhọn trong PSScriptAnalyzer, sau đó xem lại các bài kiểm thử hiện có về hành vi định dạng. Công việc được xem là hoàn tất khi một quy tắc riêng tạo ra các dòng trống được đề xuất sau dấu ngoặc nhọn và các bài kiểm thử bao quát ví dụ PowerShell lồng nhau được nê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
- Tính năng
- Độ 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
- Khá rõ ràng
- Mức phù hợp với người mới
- 35/100