From 93c23d71b9577a355a4c7f0c8fa8c3ec2c3cfe77 Mon Sep 17 00:00:00 2001
From: Amadeusz Sadowski <amadeusz.sadowski@allegro.com>
Date: Thu, 4 Jan 2024 12:18:04 +0100
Subject: [PATCH] refactor: split into common and csharp-specific

---
 .github/workflows/template.yml                |  1 +
 .gitignore                                    |  1 +
 Directory.Build.props                         |  5 +--
 README.md                                     | 15 ++++---
 global.json                                   |  5 +--
 nuget.config                                  |  6 +--
 ... Allegro.NET.Sdk.CSharp.Analyzers.targets} |  0
 .../Sdk/Allegro.NET.Sdk.CSharp.props          | 27 +++++++++++
 .../Sdk/Allegro.NET.Sdk.CSharp.targets        | 30 +++++++++++++
 .../Sdk/Allegro.NET.Sdk.Common.props          | 25 +++++++++++
 .../Sdk/Allegro.NET.Sdk.Common.targets        | 29 ++++++++++++
 .../Sdk/Allegro.NET.Sdk.props                 |  6 +++
 .../Sdk/Allegro.NET.Sdk.targets               |  6 +++
 src/Allegro.DotnetSdk/Sdk/Sdk.props           |  4 +-
 .../Sdk/UseAllegroDotnetSdkDefaults.props     | 41 -----------------
 .../Sdk/UseAllegroDotnetSdkDefaults.targets   | 45 -------------------
 16 files changed, 143 insertions(+), 103 deletions(-)
 rename src/Allegro.DotnetSdk/Sdk/{UseDefaultAnalyzers.targets => Allegro.NET.Sdk.CSharp.Analyzers.targets} (100%)
 create mode 100644 src/Allegro.DotnetSdk/Sdk/Allegro.NET.Sdk.CSharp.props
 create mode 100644 src/Allegro.DotnetSdk/Sdk/Allegro.NET.Sdk.CSharp.targets
 create mode 100644 src/Allegro.DotnetSdk/Sdk/Allegro.NET.Sdk.Common.props
 create mode 100644 src/Allegro.DotnetSdk/Sdk/Allegro.NET.Sdk.Common.targets
 create mode 100644 src/Allegro.DotnetSdk/Sdk/Allegro.NET.Sdk.props
 create mode 100644 src/Allegro.DotnetSdk/Sdk/Allegro.NET.Sdk.targets
 delete mode 100644 src/Allegro.DotnetSdk/Sdk/UseAllegroDotnetSdkDefaults.props
 delete mode 100644 src/Allegro.DotnetSdk/Sdk/UseAllegroDotnetSdkDefaults.targets

diff --git a/.github/workflows/template.yml b/.github/workflows/template.yml
index 0fd020a..7ffc8ad 100644
--- a/.github/workflows/template.yml
+++ b/.github/workflows/template.yml
@@ -33,6 +33,7 @@ jobs:
       with:
         dotnet-version: |
           6.0.x
+          8.0.x
     - name: Restore dependencies
       run: dotnet restore
     - name: Build
diff --git a/.gitignore b/.gitignore
index 05b81fb..ea39a34 100644
--- a/.gitignore
+++ b/.gitignore
@@ -1,6 +1,7 @@
 .idea/
 .ionide/
 App_Data/
+artifacts/
 packages/
 package/
 obj/
diff --git a/Directory.Build.props b/Directory.Build.props
index f927eb1..5154eb3 100644
--- a/Directory.Build.props
+++ b/Directory.Build.props
@@ -9,6 +9,7 @@
     <PackageLicenseFile>LICENSE</PackageLicenseFile>
     <PackageReadmeFile>README.md</PackageReadmeFile>
     <PublishRepositoryUrl>true</PublishRepositoryUrl>
+    <UseArtifactsOutput>true</UseArtifactsOutput>
   </PropertyGroup>
 
   <ItemGroup>
@@ -18,8 +19,4 @@
     <!-- <None Include="$(MSBuildThisFileDirectory)CHANGELOG.md" Pack="true" PackagePath=""/> -->
   </ItemGroup>
 
-  <ItemGroup>
-    <PackageReference Include="Microsoft.SourceLink.GitHub" Version="1.1.1" PrivateAssets="all" />
-  </ItemGroup>
-
 </Project>
\ No newline at end of file
diff --git a/README.md b/README.md
index 059a960..1bb0148 100644
--- a/README.md
+++ b/README.md
@@ -52,16 +52,21 @@ The `Directory.Build.props` file should be updated in order to actually import t
 
 Most properties are only set in the SDK if not configured by the project.
 
-To configure the SDK, the following properties can be added into `Directory.Build.props` or `.csproj` files:
+To configure the SDK, the following properties can be added into `Directory.Build.props` or `.*proj` files:
 
 - `AllegroDotnetSdkEnableXmlDocAdjustments` (default: true) - enable doc file generation, suppress missing-comments warning (1591) - completely in test projects, as errors otherwise (keep as warning).
-- `TreatWarningsAsErrors` (default: true in CI and Rider IDE)
-- `UseAllegroDotnetSdkDefaultAnalyzers` (default: true when no CPM) - reference and configure several external analyzers - StyleCop, AsyncFixer, Meziantou.
+- `TreatWarningsAsErrors` (default: true in CI and Rider IDE).
 - `AllegroDotnetSdkEnableGlobalEditorConfig` (default: true) - add `editorconfig.global` analyzer configuration file.
-- `AllegroDotnetSdkEnableImplicitUsingsAdjustments` (default: true) - add `System.Collections.Immutable` and remove `Microsoft.Extensions.Logging` and `System.Net.Http` implicit usings.
+- `AllegroDotnetSdkEnablePackAdjustments` (default: true) - sets PublishRepositoryUrl and enables `.snupkg` symbols.
+- `AllegroDotnetSdkEnableAzureArtifactsAdjustments` (default: false) - disables `.snupkg` and instead embeds PDBs in `.nupkg`.
+
+For `C#` only:
+
+- `AllegroDotnetSdkEnableImplicitUsingsAdjustments` (default: true for C#) - add `System.Collections.Immutable` and remove `Microsoft.Extensions.Logging` and `System.Net.Http` implicit usings.
+- `UseAllegroDotnetSdkDefaultAnalyzers` (default: true for C# when no CPM) - reference and configure several external analyzers - StyleCop, AsyncFixer, Meziantou.
 - `AllegroDotnetSdkEnableAsyncFixer` (default: `UseAllegroDotnetSdkDefaultAnalyzers`) - references AsyncFixer package.
 - `AllegroDotnetSdkEnableMeziantou` (default: `UseAllegroDotnetSdkDefaultAnalyzers`) - references Meziantou.Analyzers package.
-- `AllegroDotnetSdkEnableStyleCop` (default: `UseAllegroDotnetSdkDefaultAnalyzers`) - references StyleCop.Analyzers package.
+- `AllegroDotnetSdkEnableStyleCop` (default: `UseAllegroDotnetSdkDefaultAnalyzers`) - references StyleCop.Analyzers package and a preset `stylecop.json`.
 
 Configure in `Directory.Build.props` - repo wide:
 
diff --git a/global.json b/global.json
index f3b3a4e..e589342 100644
--- a/global.json
+++ b/global.json
@@ -1,10 +1,9 @@
 {
     "sdk": {
-      "version": "6.0.400",
+      "version": "8.0.100",
       "rollForward": "latestFeature"
     },
     "msbuild-sdks": {
-      "Microsoft.Build.CentralPackageVersions": "2.1.3",
-      "Microsoft.Build.NoTargets": "3.5.6"
+      "Microsoft.Build.NoTargets": "3.7.56"
     }
   }
\ No newline at end of file
diff --git a/nuget.config b/nuget.config
index 7d7e3a9..2c70539 100644
--- a/nuget.config
+++ b/nuget.config
@@ -1,6 +1,6 @@
 <?xml version="1.0" encoding="utf-8"?>
 <configuration>
-    <packageSources>
-        <add key="nuget.org" value="https://api.nuget.org/v3/index.json" protocolVersion="3" />
-    </packageSources>
+  <packageSources>
+    <add key="nuget.org" value="https://api.nuget.org/v3/index.json" protocolVersion="3" />
+  </packageSources>
 </configuration>
\ No newline at end of file
diff --git a/src/Allegro.DotnetSdk/Sdk/UseDefaultAnalyzers.targets b/src/Allegro.DotnetSdk/Sdk/Allegro.NET.Sdk.CSharp.Analyzers.targets
similarity index 100%
rename from src/Allegro.DotnetSdk/Sdk/UseDefaultAnalyzers.targets
rename to src/Allegro.DotnetSdk/Sdk/Allegro.NET.Sdk.CSharp.Analyzers.targets
diff --git a/src/Allegro.DotnetSdk/Sdk/Allegro.NET.Sdk.CSharp.props b/src/Allegro.DotnetSdk/Sdk/Allegro.NET.Sdk.CSharp.props
new file mode 100644
index 0000000..88a8f4c
--- /dev/null
+++ b/src/Allegro.DotnetSdk/Sdk/Allegro.NET.Sdk.CSharp.props
@@ -0,0 +1,27 @@
+<Project>
+  <!-- This file configures C#-specific options -->
+
+  <PropertyGroup>
+    <AllegroDotnetSdkEnableImplicitUsingsAdjustments Condition="'$(AllegroDotnetSdkEnableImplicitUsingsAdjustments)' == ''">true</AllegroDotnetSdkEnableImplicitUsingsAdjustments>
+    <Nullable Condition="'$(Nullable)' == ''">enable</Nullable>
+    <ImplicitUsings Condition="'$(AllegroDotnetSdkEnableImplicitUsingsAdjustments)' == 'true' And '$(ImplicitUsings)' == ''">enable</ImplicitUsings>
+    <AnalysisLevel Condition="'$(AnalysisLevel)' == ''">latest-Recommended</AnalysisLevel>
+    <EnforceCodeStyleInBuild Condition="'$(EnforceCodeStyleInBuild)' == ''">true</EnforceCodeStyleInBuild>
+  </PropertyGroup>
+
+  <!-- XML documentation adjustments -->
+  <PropertyGroup Condition="'$(AllegroDotnetSdkEnableXmlDocAdjustments)' == 'true'">
+    <!-- Never mark missing docs as errors -->
+    <!-- CS1591: Missing XML comment for publicly visible type or member -->
+    <WarningsNotAsErrors Condition="'$(GenerateDocumentationFile)' == 'true'">$(WarningsNotAsErrors);CS1591</WarningsNotAsErrors>
+  </PropertyGroup>
+
+  <!-- Exclude selected warnings in test projects -->
+  <PropertyGroup Condition="$(MSBuildProjectName.Contains(`Tests`))">
+    <!-- CA1707: Identifiers should not contain underscores -->
+    <NoWarn>$(NoWarn);CA1707</NoWarn>
+    <!-- CS1591: Missing XML comment for publicly visible type or member -->
+    <NoWarn Condition="'$(GenerateDocumentationFile)' == 'true' And '$(AllegroDotnetSdkEnableXmlDocAdjustments)' == 'true'">$(NoWarn);CS1591</NoWarn>
+  </PropertyGroup>
+
+</Project>
\ No newline at end of file
diff --git a/src/Allegro.DotnetSdk/Sdk/Allegro.NET.Sdk.CSharp.targets b/src/Allegro.DotnetSdk/Sdk/Allegro.NET.Sdk.CSharp.targets
new file mode 100644
index 0000000..532e624
--- /dev/null
+++ b/src/Allegro.DotnetSdk/Sdk/Allegro.NET.Sdk.CSharp.targets
@@ -0,0 +1,30 @@
+<Project>
+
+  <!-- Setup this SDKs defaults -->
+  <PropertyGroup>
+    <!-- Enable default analyzers if CPM is not used. -->
+    <UseAllegroDotnetSdkDefaultAnalyzers Condition="'$(UseAllegroDotnetSdkDefaultAnalyzers)' == '' And '$(ManagePackageVersionsCentrally)' != 'true'">true</UseAllegroDotnetSdkDefaultAnalyzers>
+  </PropertyGroup>
+
+  <PropertyGroup Condition="'$(UseAllegroDotnetSdkDefaultAnalyzers)' == 'true'">
+    <AllegroDotnetSdkEnableAsyncFixer Condition="'$(AllegroDotnetSdkEnableAsyncFixer)' == ''">true</AllegroDotnetSdkEnableAsyncFixer>
+    <AllegroDotnetSdkEnableMeziantou Condition="'$(AllegroDotnetSdkEnableMeziantou)' == ''">true</AllegroDotnetSdkEnableMeziantou>
+    <AllegroDotnetSdkEnableStyleCop Condition="'$(AllegroDotnetSdkEnableStyleCop)' == ''">true</AllegroDotnetSdkEnableStyleCop>
+  </PropertyGroup>
+
+  <ItemGroup Condition="'$(AllegroDotnetSdkEnableStyleCop)' == 'true'">
+    <AdditionalFiles Include="$(MSBuildThisFileDirectory)stylecop.json" Link="stylecop.json" />
+  </ItemGroup>
+
+  <ItemGroup Label="ImplicitUsings adjustments" Condition="('$(ImplicitUsings)' == 'true' Or '$(ImplicitUsings)' == 'enable') And '$(AllegroDotnetSdkEnableImplicitUsingsAdjustments)' == 'true'">
+    <!-- Add Immutable as we often use them -->
+    <Using Include="System.Collections.Immutable" />
+    <!-- Remove Logging because it conflicts with Serilog which we use by default -->
+    <Using Remove="Microsoft.Extensions.Logging" />
+    <!-- Remove Http because we want to explicitly enforce devs to think twice before using System.Net.Http -->
+    <Using Remove="System.Net.Http" />
+  </ItemGroup>
+
+  <Import Project="Allegro.NET.Sdk.CSharp.Analyzers.targets" />
+
+</Project>
\ No newline at end of file
diff --git a/src/Allegro.DotnetSdk/Sdk/Allegro.NET.Sdk.Common.props b/src/Allegro.DotnetSdk/Sdk/Allegro.NET.Sdk.Common.props
new file mode 100644
index 0000000..02a99a9
--- /dev/null
+++ b/src/Allegro.DotnetSdk/Sdk/Allegro.NET.Sdk.Common.props
@@ -0,0 +1,25 @@
+<Project>
+  <!-- This file configures language-independent options -->
+
+  <!-- Setup this SDKs defaults -->
+  <PropertyGroup>
+    <AllegroDotnetSdkEnableXmlDocAdjustments Condition="'$(AllegroDotnetSdkEnableXmlDocAdjustments)' == ''">true</AllegroDotnetSdkEnableXmlDocAdjustments>
+  </PropertyGroup>
+
+  <!-- Warnings as Errors -->
+  <PropertyGroup>
+    <!-- Some devs want Warnings as Errors in local IDE development -->
+    <TreatWarningsAsErrors Condition="'$(TreatWarningsAsErrors)' == '' And '$(BuildingByReSharper)' == 'true'">true</TreatWarningsAsErrors>
+    <!-- Enable Warnings as Errors in CI -->
+    <TreatWarningsAsErrors Condition="'$(TreatWarningsAsErrors)' == '' And '$(CI)' != ''">true</TreatWarningsAsErrors>
+    <!-- LEGACY: enable by default until people explicitly set their preferences -->
+    <TreatWarningsAsErrors Condition="'$(TreatWarningsAsErrors)' == ''">true</TreatWarningsAsErrors>
+  </PropertyGroup>
+
+  <!-- XML documentation adjustments -->
+  <PropertyGroup Condition="'$(AllegroDotnetSdkEnableXmlDocAdjustments)' == 'true'">
+    <!-- Documentation can be disabled per-project -->
+    <GenerateDocumentationFile Condition="'$(GenerateDocumentationFile)' == ''">true</GenerateDocumentationFile>
+  </PropertyGroup>
+
+</Project>
\ No newline at end of file
diff --git a/src/Allegro.DotnetSdk/Sdk/Allegro.NET.Sdk.Common.targets b/src/Allegro.DotnetSdk/Sdk/Allegro.NET.Sdk.Common.targets
new file mode 100644
index 0000000..8bea166
--- /dev/null
+++ b/src/Allegro.DotnetSdk/Sdk/Allegro.NET.Sdk.Common.targets
@@ -0,0 +1,29 @@
+<Project>
+
+  <!-- Setup this SDKs defaults -->
+  <PropertyGroup>
+    <AllegroDotnetSdkEnableGlobalEditorConfig Condition="'$(AllegroDotnetSdkEnableGlobalEditorConfig)' == ''">true</AllegroDotnetSdkEnableGlobalEditorConfig>
+    <AllegroDotnetSdkEnablePackAdjustments Condition="'$(AllegroDotnetSdkEnablePackAdjustments)' == ''">true</AllegroDotnetSdkEnablePackAdjustments>
+    <AllegroDotnetSdkEnableAzureArtifactsAdjustments Condition="'$(AllegroDotnetSdkEnableAzureArtifactsAdjustments)' == ''">false</AllegroDotnetSdkEnableAzureArtifactsAdjustments>
+  </PropertyGroup>
+
+  <!-- Configure nuget-packing for Library/Packable non-test projects -->
+  <PropertyGroup Condition="'$(AllegroDotnetSdkEnablePackAdjustments)' == 'true' And ('$(OutputType)' == 'Library' Or '$(IsPackable)' != 'false') And '$(IsTestProject)' != 'true'">
+    <PublishRepositoryUrl>true</PublishRepositoryUrl>
+    <IncludeSymbols>true</IncludeSymbols>
+    <SymbolPackageFormat>snupkg</SymbolPackageFormat>
+  </PropertyGroup>
+
+  <PropertyGroup Condition="'$(AllegroDotnetSdkEnableAzureArtifactsAdjustments)' == 'true'">
+    <!-- Embed PDBs in .nupkg because Azure Artifacts doesn't support .snupkg yet -->
+    <AllowedOutputExtensionsInPackageBuildOutputFolder>
+      $(AllowedOutputExtensionsInPackageBuildOutputFolder);.pdb
+    </AllowedOutputExtensionsInPackageBuildOutputFolder>
+    <IncludeSymbols>false</IncludeSymbols>
+  </PropertyGroup>
+
+  <ItemGroup Condition="'$(AllegroDotnetSdkEnableGlobalEditorConfig)' == 'true'">
+    <GlobalAnalyzerConfigFiles Include="$(MSBuildThisFileDirectory)editor.globalconfig" Link="editor.globalconfig" />
+  </ItemGroup>
+
+</Project>
\ No newline at end of file
diff --git a/src/Allegro.DotnetSdk/Sdk/Allegro.NET.Sdk.props b/src/Allegro.DotnetSdk/Sdk/Allegro.NET.Sdk.props
new file mode 100644
index 0000000..c33fa26
--- /dev/null
+++ b/src/Allegro.DotnetSdk/Sdk/Allegro.NET.Sdk.props
@@ -0,0 +1,6 @@
+<Project>
+
+  <Import Project="Allegro.NET.Sdk.Common.props" />
+  <Import Project="Allegro.NET.Sdk.CSharp.props" Condition="'$(MSBuildProjectExtension)' == '.csproj'" />
+
+</Project>
\ No newline at end of file
diff --git a/src/Allegro.DotnetSdk/Sdk/Allegro.NET.Sdk.targets b/src/Allegro.DotnetSdk/Sdk/Allegro.NET.Sdk.targets
new file mode 100644
index 0000000..1131e9c
--- /dev/null
+++ b/src/Allegro.DotnetSdk/Sdk/Allegro.NET.Sdk.targets
@@ -0,0 +1,6 @@
+<Project>
+
+  <Import Project="Allegro.NET.Sdk.Common.targets" />
+  <Import Project="Allegro.NET.Sdk.CSharp.targets" Condition="'$(MSBuildProjectExtension)' == '.csproj'" />
+
+</Project>
\ No newline at end of file
diff --git a/src/Allegro.DotnetSdk/Sdk/Sdk.props b/src/Allegro.DotnetSdk/Sdk/Sdk.props
index 73d08d2..df26c55 100644
--- a/src/Allegro.DotnetSdk/Sdk/Sdk.props
+++ b/src/Allegro.DotnetSdk/Sdk/Sdk.props
@@ -9,11 +9,11 @@
     <!--
       Import our custom props just after Directory.Build.props is imported, but before Microsoft.NET.Sdk.props
     -->
-    <CustomAfterDirectoryBuildProps>$(CustomAfterDirectoryBuildProps);$(MSBuildThisFileDirectory)UseAllegroDotnetSdkDefaults.props</CustomAfterDirectoryBuildProps>
+    <CustomAfterDirectoryBuildProps>$(CustomAfterDirectoryBuildProps);$(MSBuildThisFileDirectory)Allegro.NET.Sdk.props</CustomAfterDirectoryBuildProps>
     <!--
       Import our custom targets after Directory.Build.targets is imported
     -->
-    <CustomBeforeDirectoryBuildTargets>$(CustomBeforeDirectoryBuildTargets);$(MSBuildThisFileDirectory)UseAllegroDotnetSdkDefaults.targets</CustomBeforeDirectoryBuildTargets>
+    <CustomBeforeDirectoryBuildTargets>$(CustomBeforeDirectoryBuildTargets);$(MSBuildThisFileDirectory)Allegro.NET.Sdk.targets</CustomBeforeDirectoryBuildTargets>
   </PropertyGroup>
 
   <!--
diff --git a/src/Allegro.DotnetSdk/Sdk/UseAllegroDotnetSdkDefaults.props b/src/Allegro.DotnetSdk/Sdk/UseAllegroDotnetSdkDefaults.props
deleted file mode 100644
index a5a56c7..0000000
--- a/src/Allegro.DotnetSdk/Sdk/UseAllegroDotnetSdkDefaults.props
+++ /dev/null
@@ -1,41 +0,0 @@
-<Project>
-
-  <!-- Setup this SDKs defaults -->
-  <PropertyGroup>
-    <AllegroDotnetSdkEnableXmlDocAdjustments Condition="'$(AllegroDotnetSdkEnableXmlDocAdjustments)' == ''">true</AllegroDotnetSdkEnableXmlDocAdjustments>
-  </PropertyGroup>
-
-  <PropertyGroup Label="Compiler/Language Settings">
-    <LangVersion Condition="'$(LangVersion)' == ''">latest</LangVersion>
-    <Nullable Condition="'$(Nullable)' == ''">enable</Nullable>
-    <ImplicitUsings Condition="'$(ImplicitUsings)' == ''">enable</ImplicitUsings>
-  </PropertyGroup>
-
-  <PropertyGroup Label="Analysis configuration">
-    <AnalysisLevel Condition="'$(AnalysisLevel)' == ''">latest-Recommended</AnalysisLevel>
-    <EnforceCodeStyleInBuild Condition="'$(EnforceCodeStyleInBuild)' == ''">true</EnforceCodeStyleInBuild>
-    <!-- Some devs want Warnings as Errors in local IDE development -->
-    <TreatWarningsAsErrors Condition="'$(TreatWarningsAsErrors)' == '' AND '$(BuildingByReSharper)' == 'true'">true</TreatWarningsAsErrors>
-    <!-- Enable Warnings as Errors in CI -->
-    <TreatWarningsAsErrors Condition="'$(TreatWarningsAsErrors)' == '' AND '$(CI)' != ''">true</TreatWarningsAsErrors>
-    <!-- LEGACY: enable by default until people explicitly set their preferences -->
-    <TreatWarningsAsErrors Condition="'$(TreatWarningsAsErrors)' == ''">true</TreatWarningsAsErrors>
-  </PropertyGroup>
-
-  <PropertyGroup Label="XML docs" Condition="'$(AllegroDotnetSdkEnableXmlDocAdjustments)' == 'true'">
-    <!-- Documentation can be disabled per-project -->
-    <GenerateDocumentationFile Condition="'$(GenerateDocumentationFile)' == ''">true</GenerateDocumentationFile>
-    <!-- Never mark missing docs as errors -->
-    <!-- CS1591: Missing XML comment for publicly visible type or member -->
-    <WarningsNotAsErrors Condition="'$(GenerateDocumentationFile)' == 'true'">$(WarningsNotAsErrors);CS1591</WarningsNotAsErrors>
-  </PropertyGroup>
-
-  <PropertyGroup Label="Test project properties" Condition="$(MSBuildProjectName.Contains(`Tests`))">
-    <!-- Exclude selected warnings in test projects -->
-    <!-- CA1707: Identifiers should not contain underscores -->
-    <NoWarn>$(NoWarn);CA1707</NoWarn>
-    <!-- CS1591: Missing XML comment for publicly visible type or member -->
-    <NoWarn Condition="'$(GenerateDocumentationFile)' == 'true'">$(NoWarn);CS1591</NoWarn>
-  </PropertyGroup>
-
-</Project>
\ No newline at end of file
diff --git a/src/Allegro.DotnetSdk/Sdk/UseAllegroDotnetSdkDefaults.targets b/src/Allegro.DotnetSdk/Sdk/UseAllegroDotnetSdkDefaults.targets
deleted file mode 100644
index 9a763f6..0000000
--- a/src/Allegro.DotnetSdk/Sdk/UseAllegroDotnetSdkDefaults.targets
+++ /dev/null
@@ -1,45 +0,0 @@
-<Project>
-
-  <!-- Setup this SDKs defaults -->
-  <PropertyGroup>
-    <!-- Enable default analyzers if CPM is not used. -->
-    <UseAllegroDotnetSdkDefaultAnalyzers Condition="'$(UseAllegroDotnetSdkDefaultAnalyzers)' == '' AND '$(ManagePackageVersionsCentrally)' != 'true'">true</UseAllegroDotnetSdkDefaultAnalyzers>
-    <AllegroDotnetSdkEnableGlobalEditorConfig Condition="'$(AllegroDotnetSdkEnableGlobalEditorConfig)' == ''">true</AllegroDotnetSdkEnableGlobalEditorConfig>
-    <AllegroDotnetSdkEnableImplicitUsingsAdjustments Condition="'$(AllegroDotnetSdkEnableImplicitUsingsAdjustments)' == ''">true</AllegroDotnetSdkEnableImplicitUsingsAdjustments>
-  </PropertyGroup>
-
-  <PropertyGroup Condition="'$(UseAllegroDotnetSdkDefaultAnalyzers)' == 'true'">
-    <AllegroDotnetSdkEnableAsyncFixer Condition="'$(AllegroDotnetSdkEnableAsyncFixer)' == ''">true</AllegroDotnetSdkEnableAsyncFixer>
-    <AllegroDotnetSdkEnableMeziantou Condition="'$(AllegroDotnetSdkEnableMeziantou)' == ''">true</AllegroDotnetSdkEnableMeziantou>
-    <AllegroDotnetSdkEnableStyleCop Condition="'$(AllegroDotnetSdkEnableStyleCop)' == ''">true</AllegroDotnetSdkEnableStyleCop>
-  </PropertyGroup>
-
-  <PropertyGroup Condition="'$(IsPackable)' != 'false' AND '$(IsTestProject)' != 'true'">
-    <PublishRepositoryUrl>true</PublishRepositoryUrl>
-    <!-- SourceLink configuration -->
-    <EmbedUntrackedSources>true</EmbedUntrackedSources>
-    <!-- Embed PDBs in .nupkg because Azure Artifacts doesn't support .snupkg yet -->
-    <AllowedOutputExtensionsInPackageBuildOutputFolder>
-      $(AllowedOutputExtensionsInPackageBuildOutputFolder);.pdb
-    </AllowedOutputExtensionsInPackageBuildOutputFolder>
-    <IncludeSymbols>true</IncludeSymbols>
-    <SymbolPackageFormat>snupkg</SymbolPackageFormat>
-  </PropertyGroup>
-
-  <ItemGroup>
-    <AdditionalFiles Include="$(MSBuildThisFileDirectory)stylecop.json" Link="stylecop.json" />
-    <GlobalAnalyzerConfigFiles Include="$(MSBuildThisFileDirectory)editor.globalconfig" Link="editor.globalconfig" Condition="'$(AllegroDotnetSdkEnableGlobalEditorConfig)' == 'true'" />
-  </ItemGroup>
-
-  <Import Project="UseDefaultAnalyzers.targets" Condition="'$(UseAllegroDotnetSdkDefaultAnalyzers)' == 'true'" />
-
-  <ItemGroup Label="ImplicitUsings adjustments" Condition="'$(AllegroDotnetSdkEnableImplicitUsingsAdjustments)' == 'true'">
-    <!-- Add Immutable as we often use them -->
-    <Using Include="System.Collections.Immutable" />
-    <!-- Remove Logging because it conflicts with Serilog which we use by default -->
-    <Using Remove="Microsoft.Extensions.Logging" />
-    <!-- Remove Http because we want to explicitly enforce devs to think twice before using System.Net.Http -->
-    <Using Remove="System.Net.Http" />
-  </ItemGroup>
-
-</Project>
\ No newline at end of file