Integrate hierarchyid into the EF Core codebase

Resolves efcore/EFCore.SqlServer.HierarchyId#113, resolves efcore/EFCore.SqlServer.HierarchyId#119, resolves #365

eng/helix.proj

@@ -12,7 +12,7 @@
 
     <EnableAzurePipelinesReporter>true</EnableAzurePipelinesReporter>
     <FailOnTestFailure>true</FailOnTestFailure>
-    <SqlServerTests>$(RepoRoot)/test/EFCore.SqlServer.FunctionalTests/*.csproj;$(RepoRoot)/test/EFCore.OData.FunctionalTests/*.csproj;$(RepoRoot)/test/EFCore.AspNet.SqlServer.FunctionalTests/*.csproj</SqlServerTests>
+    <SqlServerTests>$(RepoRoot)/test/EFCore.SqlServer.FunctionalTests/*.csproj;$(RepoRoot)/test/EFCore.SqlServer.HierarchyId.Tests/*.csproj;$(RepoRoot)/test/EFCore.OData.FunctionalTests/*.csproj;$(RepoRoot)/test/EFCore.AspNet.SqlServer.FunctionalTests/*.csproj</SqlServerTests>
   </PropertyGroup>
 
   <PropertyGroup Condition = "'$(SYSTEM_ACCESSTOKEN)' == ''">

src/EFCore.Abstractions/ChangeTracking/Internal/SortableBindingList.cs

@@ -43,8 +43,8 @@ public SortableBindingList(List<T> list)
         "ReflectionAnalysis",
         "IL2046",
         Justification =
-            "This method is an override, and the base method isn't annotated with RequiresUnreferencedCode. " +
-            "The entire type is marked with RequiresUnreferencedCode.")]
+            "This method is an override, and the base method isn't annotated with RequiresUnreferencedCode. "
+            + "The entire type is marked with RequiresUnreferencedCode.")]
     protected override void ApplySortCore(PropertyDescriptor prop, ListSortDirection direction)
     {
         if (PropertyComparer.CanSort(prop.PropertyType))

src/EFCore.Abstractions/ChangeTracking/ObservableCollectionListSource.cs

@@ -80,8 +80,8 @@ bool IListSource.ContainsListCollection
         "ReflectionAnalysis",
         "IL2046",
         Justification =
-            "This method is an interface implementation, and the interface method isn't annotated with RequiresUnreferencedCode. " +
-            "The entire type is marked with RequiresUnreferencedCode.")]
+            "This method is an interface implementation, and the interface method isn't annotated with RequiresUnreferencedCode. "
+            + "The entire type is marked with RequiresUnreferencedCode.")]
     IList IListSource.GetList()
         => _bindingList ??= this.ToBindingList();
 }

src/EFCore.SqlServer.Abstractions/EFCore.SqlServer.Abstractions.csproj

@@ -1,16 +1,22 @@
 <Project Sdk="Microsoft.NET.Sdk">
 
   <PropertyGroup>
-    <TargetFramework>netstandard2.0</TargetFramework>
-    <AssemblyName>EntityFrameworkCore.SqlServer.HierarchyId.Abstractions</AssemblyName>
+    <TargetFramework>netstandard2.1</TargetFramework>
+    <MinClientVersion>3.6</MinClientVersion>
+    <AssemblyName>Microsoft.EntityFrameworkCore.SqlServer.Abstractions</AssemblyName>
     <RootNamespace>Microsoft.EntityFrameworkCore</RootNamespace>
-    <Description>Common abstractions for using hierarchyid with EF Core</Description>
-    <GeneratePackageOnBuild>true</GeneratePackageOnBuild>
+    <Description>Provides abstractions that are used by models in conjunction with the SQL Server EF Core provider
+
+      Commonly Used Types:
+      Microsoft.EntityFrameworkCore.HierarchyId
+    </Description>
     <GenerateDocumentationFile>true</GenerateDocumentationFile>
+    <PackageTags>$(PackageTags);SQL Server;HierarchyId</PackageTags>
+    <ImplicitUsings>true</ImplicitUsings>
   </PropertyGroup>
 
   <ItemGroup>
-    <PackageReference Include="dotMorten.Microsoft.SqlServer.Types" Version="1.5.0" />
+    <PackageReference Include="Microsoft.SqlServer.Types" Version="160.1000.6" />
   </ItemGroup>
 
 </Project>