Merge pull request #1170 from CommunityToolkit/dev/reproducible-builds

Fix deterministic builds, improve GitHub links

Author: Sergio0694

Directory.Build.props

@@ -24,17 +24,19 @@
     </Otherwise>
   </Choose>
 
+  <!-- Build-only packages for versioning and reproducible builds -->
   <ItemGroup>
     <PackageReference Include="Nerdbank.GitVersioning" Version="3.9.50" PrivateAssets="All" IsImplicitlyReferenced="true" />
+    <PackageReference Include="DotNet.ReproducibleBuilds" Version="2.0.2" PrivateAssets="all" IsImplicitlyReferenced="true" />
   </ItemGroup>
 
   <Choose>
     <When Condition="'$(SourceLinkEnabled)' != 'false'">
       <PropertyGroup>
-        <!-- Declare that the Repository URL can be published to NuSpec -->
-        <PublishRepositoryUrl>true</PublishRepositoryUrl>
+
         <!-- Embed source files that are not tracked by the source control manager to the PDB -->
         <EmbedUntrackedSources>true</EmbedUntrackedSources>
+
         <!-- Include PDB in the built .nupkg -->
         <AllowedOutputExtensionsInPackageBuildOutputFolder>$(AllowedOutputExtensionsInPackageBuildOutputFolder);.pdb</AllowedOutputExtensionsInPackageBuildOutputFolder>
       </PropertyGroup>

build/Community.Toolkit.Common.props

@@ -46,7 +46,6 @@
   <PropertyGroup>
     <IsPackable>true</IsPackable>
     <IsPublishable>true</IsPublishable>
-    <ContinuousIntegrationBuild>$(TF_BUILD)</ContinuousIntegrationBuild>
   </PropertyGroup>
 
 </Project>

src/CommunityToolkit.Mvvm.CodeFixers/CommunityToolkit.Mvvm.CodeFixers.props

@@ -4,9 +4,6 @@
     <TargetFramework>netstandard2.0</TargetFramework>
     <IsPackable>false</IsPackable>
     <EnforceExtendedAnalyzerRules>true</EnforceExtendedAnalyzerRules>
-
-    <!-- Embed the .pdb file (see notes in the MVVM Toolkit generator projects) -->
-    <DebugType>embedded</DebugType>
   </PropertyGroup>
 
   <!-- Simplified version of the same props as the source generators -->

src/CommunityToolkit.Mvvm.SourceGenerators/CommunityToolkit.Mvvm.SourceGenerators.props

@@ -4,15 +4,6 @@
     <TargetFramework>netstandard2.0</TargetFramework>
     <IsPackable>false</IsPackable>
     <EnforceExtendedAnalyzerRules>true</EnforceExtendedAnalyzerRules>
-
-    <!--
-      While this project does not produce a .dll that consuming projects execute at runtime, we still want to
-      include debugging information, to make it easier to debug the generator themselves in case of issues.
-      Rather than using portable .pdb files, we just embed them, to ensure debuggers will load them correctly.
-      Using portable .pdb-s results in Visual Studio not automatically loading them when attaching a debugger
-      to the Roslyn code analysis process to try to debug the generators, forcing to pick the path manually.
-    -->
-    <DebugType>embedded</DebugType>
   </PropertyGroup>
 
   <!--