PowerShell / PowerShell/PSScriptAnalyzer
Diagnostics.CodeAnalysis.SuppressMessageAttribute should take an array of category names
未關閉
@kapilmb 已經在處理了。
開始於 2016年3月7日。
Issue - Enhancement
- 主要語言
- C#
- 星號
- 2.2k
- 分支
- 414
- 平均合併
- 13 小時 1 分鐘
- 30 天內合併 PR
- 2
描述
The attribute needs to take an array of categories. I shouldn't have to add a new instance of the attribute for each category I want to suppress.
Having to write a separate line for every rule I need to suppress is extremely frustrating and verbose. Take for example this 2 line internal function:
function New-PSCredential {
[Diagnostics.CodeAnalysis.SuppressMessageAttribute("PSAvoidUsingPlainTextForPassword")] # It's encoded, leave me alone
[Diagnostics.CodeAnalysis.SuppressMessageAttribute("PSAvoidUsingUserNameAndPasswordParams")] # That's the whole point of this function
[Diagnostics.CodeAnalysis.SuppressMessageAttribute("PSProvideCommentHelp")] # No point for a constructor function with parameters this simple
[Diagnostics.CodeAnalysis.SuppressMessageAttribute("PSProvideDefaultParameterValue")] # No point for internal functions
[Diagnostics.CodeAnalysis.SuppressMessageAttribute("PSUseOutputTypeCorrectly")] # No point for internal functions
param([string]$UserName, [string]$EncodedPassword)
New-Object PSCredential $UserName, (ConvertTo-SecureString $EncodedPassword)
}
Seriously?
貢獻指南
從這裡開始
- 先讀完整個 Issue,再讀專案的貢獻指南。
- 在 Issue 下留言說明你要接手 —— 這能避免兩個人做同樣的事。
- Fork 儲存庫,在一個分支上完成修改。
- 送出 Pull Request,並在描述裡引用這個 Issue 編號。
評估
這個 Issue 還沒有評估資料。