PowerShell / PowerShell/PSScriptAnalyzer

Diagnostics.CodeAnalysis.SuppressMessageAttribute should take an array of category names

未關閉
#459 2 則留言 4 個 reaction 已指派 1 人 在 GitHub 檢視

@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?

貢獻指南

開啟貢獻指南

從這裡開始

  1. 先讀完整個 Issue,再讀專案的貢獻指南。
  2. 在 Issue 下留言說明你要接手 —— 這能避免兩個人做同樣的事。
  3. Fork 儲存庫,在一個分支上完成修改。
  4. 送出 Pull Request,並在描述裡引用這個 Issue 編號。

評估

這個 Issue 還沒有評估資料。

把新 issue 寄到你的電子郵件信箱

精選適合新手參與的 GitHub issue 摘要。