PowerShell / PowerShell/PSScriptAnalyzer
Diagnostics.CodeAnalysis.SuppressMessageAttribute should take an array of category names
Abierto
@kapilmb ya está trabajando en esto.
Desde el 7/3/2016.
Issue - Enhancement
- Lenguaje dominante
- C#
- Estrellas
- 2.2k
- Forks
- 414
- Merge medio
- 13 h 1 min
- PR fusionados (30 d)
- 2
Descripción
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?
Guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Evaluación
Este issue todavía no se ha evaluado.