From 644345306761f1ef632a7d4c3c1f5612bf448ebd Mon Sep 17 00:00:00 2001
From: Eddy Nakamura <eddynaka@gmail.com>
Date: Tue, 10 Nov 2020 15:24:06 -0300
Subject: [PATCH] Enable Github sourcelink (#2148)

* Enable Github sourcelink

* adding deterministic

* updating buildprops

* fixing

* testing nuspec

* trying to fix base path

* changing to reporoot

* rollback changes

* removing unstable bypass

* updating ado-build script

* removing property from sarif.csproj and updating releasehistory
---
 ado-build.yml          |  2 +-
 src/ReleaseHistory.md  |  1 +
 src/Sarif/Sarif.csproj |  2 +-
 src/build.props        | 27 ++++++++++++++++++++++-----
 4 files changed, 25 insertions(+), 7 deletions(-)

diff --git a/ado-build.yml b/ado-build.yml
index d807edc88..7f6132193 100644
--- a/ado-build.yml
+++ b/ado-build.yml
@@ -1,5 +1,5 @@
 pr:
-- '*'
+- 'master'
 
 pool:
   vmImage: 'windows-latest'
diff --git a/src/ReleaseHistory.md b/src/ReleaseHistory.md
index 32c4d729b..330c35c9e 100644
--- a/src/ReleaseHistory.md
+++ b/src/ReleaseHistory.md
@@ -2,6 +2,7 @@
 
 * FEATURE: PACKAGE BREAKING: Upgrade from .NET Framework 4.5 to .NET Framework 4.5.2 [#2135](https://github.com/microsoft/sarif-sdk/pull/2135)
 * FEATURE: Multitool SARIF merge accepts `threads` parameter [#2026](https://github.com/microsoft/sarif-sdk/pull/2026)
+* FEATURE: Enable GitHub SourceLink to SDK project [#2148](https://github.com/microsoft/sarif-sdk/pull/2148)
 
 ## **v2.3.7** [Sdk](https://www.nuget.org/packages/Sarif.Sdk/2.3.7) | [Driver](https://www.nuget.org/packages/Sarif.Driver/2.3.7) | [Converters](https://www.nuget.org/packages/Sarif.Converters/2.3.7) | [Multitool](https://www.nuget.org/packages/Sarif.Multitool/2.3.7) | [Multitool Library](https://www.nuget.org/packages/Sarif.Multitool.Library/2.3.7)
 * DEPENDENCY BREAKING: SARIF now requires Newtonsoft.JSON 11.0.2 (rather than 10.0.3)
diff --git a/src/Sarif/Sarif.csproj b/src/Sarif/Sarif.csproj
index 8c9a344ad..d1a220f88 100644
--- a/src/Sarif/Sarif.csproj
+++ b/src/Sarif/Sarif.csproj
@@ -32,6 +32,7 @@
   <ItemGroup>
     <PackageReference Include="Newtonsoft.Json" Version="10.0.3" />
     <PackageReference Include="System.Collections.Immutable" Version="1.5.0" />
+    <PackageReference Include="Microsoft.SourceLink.GitHub" Version="1.0.0" PrivateAssets="All"/>
   </ItemGroup>
 
   <ItemGroup>
@@ -96,5 +97,4 @@
 
   <Import Project="ToDotNet\ToDotNet.targets" />
 
-  <ProjectExtensions><VisualStudio><UserProperties CodeGenHints_1json__JSONSchema="http://json.schemastore.org/asmdef" /></VisualStudio></ProjectExtensions>
 </Project>
diff --git a/src/build.props b/src/build.props
index fd35e5c2d..f5d7c912b 100644
--- a/src/build.props
+++ b/src/build.props
@@ -11,7 +11,7 @@
     <Company Condition=" '$(Company)' == '' ">Microsoft</Company>
     <Product Condition=" '$(Product)' == '' ">Microsoft SARIF SDK</Product>
     <Copyright Condition=" '$(Copyright)' == '' ">© Microsoft Corporation. All rights reserved.</Copyright>
-    
+
     <!--
     VersionPrefix denotes the current version of the SDK. You must update it before every nuget drop.
 
@@ -24,9 +24,9 @@
     <VersionPrefix>2.3.8</VersionPrefix>
     <PreviousVersionPrefix>2.3.7</PreviousVersionPrefix>
 
-     <!-- SchemaVersionAsPublishedToSchemaStoreOrg identifies the current published version on json schema store at https://schemastore.azurewebsites.net/schemas/json/ -->
+    <!-- SchemaVersionAsPublishedToSchemaStoreOrg identifies the current published version on json schema store at https://schemastore.azurewebsites.net/schemas/json/ -->
     <SchemaVersionAsPublishedToSchemaStoreOrg>2.1.0-rtm.5</SchemaVersionAsPublishedToSchemaStoreOrg>
-      
+
     <!-- The stable version of the SARIF specification. -->
     <StableSarifVersion>2.1.0</StableSarifVersion>
 
@@ -48,7 +48,7 @@
     <ErrorReport>prompt</ErrorReport>
     <WarningLevel>4</WarningLevel>
     <TreatWarningsAsErrors>true</TreatWarningsAsErrors>
-    <HighEntropyVA>true</HighEntropyVA> 
+    <HighEntropyVA>true</HighEntropyVA>
     <!-- The line causes the assemblies in a project's referenced NuGet packages to be
          copied to the output directory. If we omit it from a test project, the tests
          fail on AppVeyor (and only there, for some reason).
@@ -64,7 +64,7 @@
     <DefineConstants>DEBUG;TRACE;CODE_ANALYSIS</DefineConstants>
   </PropertyGroup>
   <PropertyGroup Condition=" '$(Configuration)' == 'Release' "
-                 Label="Release build">
+               Label="Release build">
     <DebugType>portable</DebugType>
     <Optimize>true</Optimize>
     <DefineConstants>TRACE</DefineConstants>
@@ -96,6 +96,13 @@
     See https://github.com/NuGet/Home/issues/4687#issuecomment-393302779.
     -->
     <NoWarn>NU5105</NoWarn>
+
+    <PublishRepositoryUrl>true</PublishRepositoryUrl>
+    <EmbedUntrackedSources>true</EmbedUntrackedSources>
+    <IncludeSymbols>true</IncludeSymbols>
+    <SymbolPackageFormat>snupkg</SymbolPackageFormat>
+    <RepositoryType>git</RepositoryType>
+    <RepositoryUrl>https://github.com/Microsoft/sarif-sdk</RepositoryUrl>
   </PropertyGroup>
 
   <ItemGroup>
@@ -106,4 +113,14 @@
     <Compile Include="$(MsBuildThisFileDirectory)Shared\CommonAssemblyInfo.cs" />
   </ItemGroup>
 
+  <PropertyGroup Condition=" '$(Configuration)' == 'Release' ">
+    <ContinuousIntegrationBuild>true</ContinuousIntegrationBuild>
+    <TargetFrameworkMonikerAssemblyAttributesPath>$([System.IO.Path]::Combine('$(IntermediateOutputPath)','$(TargetFrameworkMoniker).AssemblyAttributes$(DefaultLanguageSourceExtension)'))</TargetFrameworkMonikerAssemblyAttributesPath>
+    <Deterministic>true</Deterministic>
+  </PropertyGroup>
+  <ItemGroup Condition=" '$(Configuration)' == 'Release' ">
+    <EmbeddedFiles Include="$(GeneratedAssemblyInfoFile)"/>
+    <SourceRoot Include="$(MSBuildThisFileDirectory)/" />
+  </ItemGroup>
+
 </Project>