Revert "Try out build with '$(DotNetUseShippingVersions)' == 'true'"

- didn't improve template test results

This reverts commit 9fe1c49.

Author: dougbu

Directory.Build.props

@@ -22,12 +22,6 @@
     <IsAnalyzersProject Condition="$(MSBuildProjectName.EndsWith('.Analyzers'))">true</IsAnalyzersProject>
     <IsShipping Condition="'$(IsSampleProject)' == 'true' or '$(IsTestAssetProject)' == 'true' or '$(IsBenchmarkProject)' == 'true' or '$(IsUnitTestProject)' == 'true'">false</IsShipping>
 
-    <!--
-      Disable Arcade version overrides entirely. This produces non-deterministic builds but reduces the need for
-      workarounds for assembly and file versions.
-    -->
-    <DotNetUseShippingVersions Condition=" '$(Language)' == 'C#' ">true</DotNetUseShippingVersions>
-
     <!--
       Following logic mimics core-setup approach as well as
       https://github.com/dotnet/arcade/blob/694d59f090b743f894779d04a7ffe11cbaf352e7/src/Microsoft.DotNet.Arcade.Sdk/tools/Publish.proj#L30-L31