microsoft / microsoft-performance-toolkit-sdk

Software Development Kit for the Microsoft Performance ToolKit
MIT License
156 stars 55 forks source link

PluginsSystem: consolidate plugin information into PluginInfo class #310

Closed mslukebo closed 1 year ago

mslukebo commented 1 year ago

AvailablePluginInfo, InstalledPluginInfo, and PluginMetadata was declaring identical properties, leading to duplicated code and sources of truth. This PR consolidates these properties into a PluginInfo class that is exposed on AvailablePluginInfo and InstalledPluginInfo.

Instead of also exposing this class on PluginMetadata, this PR also adds a new file to plugin packages that contains a serialization of a plugin's PluginInfo.

Helen's update 6/30 Updated property names based on PR comments. Renamed PluginInfo to PluginMetadata, and PluginContentsInfo to PluginContentsMetadata. Moved Metadata folder out of Packaging.

helenkzhang commented 1 year ago

Updated property names based on PR comments. Renamed PluginInfo to PluginMetadata, and PluginContentsInfo to PluginContentsMetadata. Moved Metadata folder out of Packaging.