Fix MessageResourceNames merging

microsoft / sarif-sdk

.NET code and supporting files for working with the 'Static Analysis Results Interchange Format' (SARIF, see https://github.com/oasis-tcs/sarif-spec)

Other

189 stars 88 forks source link

Fix MessageResourceNames merging #2793

Closed EasyRhinoMSFT closed 4 months ago

EasyRhinoMSFT commented 4 months ago

The previous architecture seemed to be sort of incompatible with the existing skimmer platform in some cases. This change sets up lists of message strings in the rule and base classes, with the Rule.MessageResourceNames property returning a merged list. It also cleans up some overlooked messiness.

EasyRhinoMSFT commented 4 months ago

Anyone else getting an error when you open this in CodeFlow?