From aba4217a07d18f30ec362fa60eab4bdc499c76b9 Mon Sep 17 00:00:00 2001
From: Cijo Thomas <cithomas@microsoft.com>
Date: Fri, 16 Apr 2021 13:40:49 -0700
Subject: [PATCH] Remove netcoreapp2.1 from logging examples (#1993)

---
 docs/Directory.Build.props                     |  2 +-
 docs/logs/extending-the-sdk/Program.cs         | 12 ------------
 .../extending-the-sdk/extending-the-sdk.csproj |  7 +------
 docs/logs/getting-started/Program.cs           | 13 -------------
 docs/logs/getting-started/README.md            | 18 ++----------------
 .../getting-started/getting-started.csproj     |  7 +------
 6 files changed, 5 insertions(+), 54 deletions(-)

diff --git a/docs/Directory.Build.props b/docs/Directory.Build.props
index 230f010de60..95ae4329f8e 100644
--- a/docs/Directory.Build.props
+++ b/docs/Directory.Build.props
@@ -4,7 +4,7 @@
   <PropertyGroup>
     <OutputType>Exe</OutputType>
     <!-- https://dotnet.microsoft.com/download/dotnet-core -->
-    <TargetFrameworks>netcoreapp2.1;netcoreapp3.1;net5.0</TargetFrameworks>
+    <TargetFrameworks>netcoreapp3.1;net5.0</TargetFrameworks>
     <!-- https://dotnet.microsoft.com/download/dotnet-framework -->
     <TargetFrameworks Condition="$(OS) == 'Windows_NT'">$(TargetFrameworks);net461;net462;net47;net471;net472;net48</TargetFrameworks>
   </PropertyGroup>
diff --git a/docs/logs/extending-the-sdk/Program.cs b/docs/logs/extending-the-sdk/Program.cs
index e47cb304ac2..4da7a2a87bb 100644
--- a/docs/logs/extending-the-sdk/Program.cs
+++ b/docs/logs/extending-the-sdk/Program.cs
@@ -15,9 +15,6 @@
 // </copyright>
 
 using System.Collections.Generic;
-#if NETCOREAPP2_1
-using Microsoft.Extensions.DependencyInjection;
-#endif
 using Microsoft.Extensions.Logging;
 using OpenTelemetry;
 using OpenTelemetry.Logs;
@@ -26,11 +23,7 @@ public class Program
 {
     public static void Main()
     {
-#if NETCOREAPP2_1
-        var serviceCollection = new ServiceCollection().AddLogging(builder =>
-#else
         using var loggerFactory = LoggerFactory.Create(builder =>
-#endif
             builder.AddOpenTelemetry(options =>
             {
                 options.IncludeScopes = true;
@@ -40,12 +33,7 @@ public static void Main()
                        .AddMyExporter();
             }));
 
-#if NETCOREAPP2_1
-        using var serviceProvider = serviceCollection.BuildServiceProvider();
-        var logger = serviceProvider.GetRequiredService<ILogger<Program>>();
-#else
         var logger = loggerFactory.CreateLogger<Program>();
-#endif
 
         // unstructured log
         logger.LogInformation("Hello, World!");
diff --git a/docs/logs/extending-the-sdk/extending-the-sdk.csproj b/docs/logs/extending-the-sdk/extending-the-sdk.csproj
index aed90386e92..a5ae2beeec6 100644
--- a/docs/logs/extending-the-sdk/extending-the-sdk.csproj
+++ b/docs/logs/extending-the-sdk/extending-the-sdk.csproj
@@ -3,12 +3,7 @@
     <ProjectReference Include="$(RepoRoot)\src\OpenTelemetry\OpenTelemetry.csproj" />
   </ItemGroup>
 
-  <ItemGroup Condition="'$(TargetFramework)' == 'netcoreapp2.1'">
-    <PackageReference Include="Microsoft.Extensions.Logging" Version="2.1.1" />
-    <PackageReference Include="Microsoft.Extensions.DependencyInjection" Version="2.1.1" />
-  </ItemGroup>
-
-  <ItemGroup Condition="'$(TargetFramework)' != 'netcoreapp2.1'">
+  <ItemGroup>
     <PackageReference Include="Microsoft.Extensions.Logging" Version="$(MicrosoftExtensionsLoggingPkgVer)" />
   </ItemGroup>
 </Project>
diff --git a/docs/logs/getting-started/Program.cs b/docs/logs/getting-started/Program.cs
index 57fc6616e27..e0eb8d41781 100644
--- a/docs/logs/getting-started/Program.cs
+++ b/docs/logs/getting-started/Program.cs
@@ -14,9 +14,6 @@
 // limitations under the License.
 // </copyright>
 
-#if NETCOREAPP2_1
-using Microsoft.Extensions.DependencyInjection;
-#endif
 using Microsoft.Extensions.Logging;
 using OpenTelemetry.Logs;
 
@@ -24,23 +21,13 @@ public class Program
 {
     public static void Main()
     {
-#if NETCOREAPP2_1
-        var serviceCollection = new ServiceCollection().AddLogging(builder =>
-#else
         using var loggerFactory = LoggerFactory.Create(builder =>
-#endif
         {
             builder.AddOpenTelemetry(options => options
                 .AddConsoleExporter());
         });
 
-#if NETCOREAPP2_1
-        using var serviceProvider = serviceCollection.BuildServiceProvider();
-        var logger = serviceProvider.GetRequiredService<ILogger<Program>>();
-#else
         var logger = loggerFactory.CreateLogger<Program>();
-#endif
-
         logger.LogInformation("Hello from {name} {price}.", "tomato", 2.99);
     }
 }
diff --git a/docs/logs/getting-started/README.md b/docs/logs/getting-started/README.md
index a5898c96312..0f4f38f8250 100644
--- a/docs/logs/getting-started/README.md
+++ b/docs/logs/getting-started/README.md
@@ -17,21 +17,7 @@ You should see the following output:
 Hello World!
 ```
 
-Let's install the packages to get started with logging. If your ConsoleApp
-targets `netcoreapp2.1` install the version `2.1.1` of
-`Microsoft.Extensions.Logging` and `Microsoft.Extensions.DependencyInjection`
-packages:
-
-  ```sh
-  dotnet add package Microsoft.Extensions.Logging -v 2.1.1
-  ```
-
-  ```sh
-  dotnet add package Microsoft.Extensions.DependencyInjection -v 2.1.1
-  ```
-
-If your ConsoleApp targets a framework other than `netcoreapp2.1`, install the
-latest Microsoft.Extensions.Logging package:
+Install the latest `Microsoft.Extensions.Logging` package:
 
   ```sh
   dotnet add package Microsoft.Extensions.Logging
@@ -39,7 +25,7 @@ latest Microsoft.Extensions.Logging package:
 
 Install the
 [OpenTelemetry.Exporter.Console](../../../src/OpenTelemetry.Exporter.Console/README.md)
-package (Use the latest version):
+package:
 
 ```sh
 dotnet add package OpenTelemetry.Exporter.Console
diff --git a/docs/logs/getting-started/getting-started.csproj b/docs/logs/getting-started/getting-started.csproj
index 4db489c913b..93374308e63 100644
--- a/docs/logs/getting-started/getting-started.csproj
+++ b/docs/logs/getting-started/getting-started.csproj
@@ -3,12 +3,7 @@
     <ProjectReference Include="$(RepoRoot)\src\OpenTelemetry.Exporter.Console\OpenTelemetry.Exporter.Console.csproj" />
   </ItemGroup>
 
-  <ItemGroup Condition="'$(TargetFramework)' == 'netcoreapp2.1'">
-    <PackageReference Include="Microsoft.Extensions.Logging" Version="2.1.1" />
-    <PackageReference Include="Microsoft.Extensions.DependencyInjection" Version="2.1.1" />
-  </ItemGroup>
-
-  <ItemGroup Condition="'$(TargetFramework)' != 'netcoreapp2.1'">
+  <ItemGroup>
     <PackageReference Include="Microsoft.Extensions.Logging" Version="$(MicrosoftExtensionsLoggingPkgVer)" />
   </ItemGroup>
 </Project>