Avoid using Type in IViewModelMetadata

We don't want to pre-load all referenced assemblies.

Author: jcansdale

src/GitHub.App/Factories/ViewViewModelFactory.cs

@@ -43,7 +43,7 @@ public FrameworkElement CreateView<TViewModel>() where TViewModel : IViewModel
         /// <inheritdoc/>
         public FrameworkElement CreateView(Type viewModel)
         {
-            var f = Views.FirstOrDefault(x => x.Metadata.ViewModelType.Contains(viewModel));
+            var f = Views.FirstOrDefault(x => x.Metadata.ViewModelType.Contains(viewModel.FullName));
             return f?.CreateExport().Value;
         }
     }

src/GitHub.Exports/Exports/ExportMetadata.cs

@@ -37,10 +37,10 @@ public sealed class ExportViewForAttribute : ExportAttribute
         public ExportViewForAttribute(Type viewModelType)
             : base(typeof(FrameworkElement))
         {
-            ViewModelType = viewModelType;
+            ViewModelType = viewModelType.FullName;
         }
 
-        public Type ViewModelType { get; }
+        public string ViewModelType { get; }
     }
 
     /// <summary>
@@ -69,7 +69,7 @@ public ExportMenuAttribute() : base(typeof(IMenuHandler))
     public interface IViewModelMetadata
     {
         [SuppressMessage("Microsoft.Performance", "CA1819:PropertiesShouldNotReturnArrays")]
-        Type[] ViewModelType { get; }
+        string[] ViewModelType { get; }
     }
 
     /// <summary>