Also catch case where the recommendation is empty string (#532)

* Try fix missing Help.Text parameter when Recommendation and Description are both empty

* More comprehensive fix
This commit is contained in:
Gabe Stocco 2023-05-25 00:53:57 +00:00 коммит произвёл GitHub
Родитель bd2e0f8bd3
Коммит 95bfa6d982
Не найден ключ, соответствующий данной подписи
Идентификатор ключа GPG: 4AEE18F83AFDEB23
1 изменённых файлов: 3 добавлений и 1 удалений

Просмотреть файл

@ -202,7 +202,9 @@ namespace Microsoft.DevSkim.CLI.Writers
sarifRule.FullDescription = new MultiformatMessageString() { Text = devskimRule.Description };
sarifRule.Help = new MultiformatMessageString()
{
Text = devskimRule.Recommendation ?? devskimRule.Description ?? $"Visit {helpUri} for guidance on this issue.",
// If recommendation is present use that, otherwise use description if present, otherwise use the HelpUri
Text = !string.IsNullOrEmpty(devskimRule.Recommendation) ? devskimRule.Recommendation :
(!string.IsNullOrEmpty(devskimRule.Description) ? devskimRule.Description : $"Visit {helpUri} for guidance on this issue."),
Markdown = $"Visit [{helpUri}]({helpUri}) for guidance on this issue."
};
sarifRule.HelpUri = helpUri;