From bad7fa7a99d0e97a06ff0989bc681dbf7027e57c Mon Sep 17 00:00:00 2001
From: jsonbailey <jbailey@launchdarkly.com>
Date: Mon, 2 Jun 2025 13:13:39 +0000
Subject: [PATCH 1/3] chore: Identify release please version from config

---
 .../client/src/LaunchDarkly.ClientSdk.csproj  |  3 -
 .../src/LaunchDarkly.ServerSdk.Ai.csproj      |  2 -
 .../server/src/LaunchDarkly.ServerSdk.csproj  |  2 -
 .../LaunchDarkly.ServerSdk.Telemetry.csproj   |  2 -
 release-please-config.json                    | 60 ++++++++++++++++---
 5 files changed, 51 insertions(+), 18 deletions(-)

diff --git a/pkgs/sdk/client/src/LaunchDarkly.ClientSdk.csproj b/pkgs/sdk/client/src/LaunchDarkly.ClientSdk.csproj
index 3fd16c47..abdedddf 100644
--- a/pkgs/sdk/client/src/LaunchDarkly.ClientSdk.csproj
+++ b/pkgs/sdk/client/src/LaunchDarkly.ClientSdk.csproj
@@ -1,9 +1,6 @@
 <Project Sdk="Microsoft.NET.Sdk">
-
     <PropertyGroup>
-        <!--x-release-please-start-version-->
         <Version>5.4.0</Version>
-        <!--x-release-please-end-->
         <!-- The BUILDFRAMEWORKS variable allows us to override the target frameworks in
              a situation where we need to build only the .NET Standard target and do not
              want to even mention the Android/iOS/Mac/Windows targets, because we're in an environment
diff --git a/pkgs/sdk/server-ai/src/LaunchDarkly.ServerSdk.Ai.csproj b/pkgs/sdk/server-ai/src/LaunchDarkly.ServerSdk.Ai.csproj
index 4ebabad3..3ed3edab 100644
--- a/pkgs/sdk/server-ai/src/LaunchDarkly.ServerSdk.Ai.csproj
+++ b/pkgs/sdk/server-ai/src/LaunchDarkly.ServerSdk.Ai.csproj
@@ -1,8 +1,6 @@
 <Project Sdk="Microsoft.NET.Sdk">
   <PropertyGroup>
-    <!--x-release-please-start-version-->
     <Version>0.7.0</Version>
-    <!--x-release-please-end-->
     <!-- The BUILDFRAMEWORKS variable allows us to override the target frameworks with a
          single framework that we are testing; this allows us to test with older SDK
          versions that would error out if they saw any newer target frameworks listed
diff --git a/pkgs/sdk/server/src/LaunchDarkly.ServerSdk.csproj b/pkgs/sdk/server/src/LaunchDarkly.ServerSdk.csproj
index 42d55d47..4c4e49fd 100644
--- a/pkgs/sdk/server/src/LaunchDarkly.ServerSdk.csproj
+++ b/pkgs/sdk/server/src/LaunchDarkly.ServerSdk.csproj
@@ -1,8 +1,6 @@
 <Project Sdk="Microsoft.NET.Sdk">
   <PropertyGroup>
-    <!--x-release-please-start-version-->
     <Version>8.8.0</Version>
-    <!--x-release-please-end-->
     <!-- The BUILDFRAMEWORKS variable allows us to override the target frameworks with a
          single framework that we are testing; this allows us to test with older SDK
          versions that would error out if they saw any newer target frameworks listed
diff --git a/pkgs/telemetry/src/LaunchDarkly.ServerSdk.Telemetry.csproj b/pkgs/telemetry/src/LaunchDarkly.ServerSdk.Telemetry.csproj
index 0902ecec..eb08bf60 100644
--- a/pkgs/telemetry/src/LaunchDarkly.ServerSdk.Telemetry.csproj
+++ b/pkgs/telemetry/src/LaunchDarkly.ServerSdk.Telemetry.csproj
@@ -1,8 +1,6 @@
 <Project Sdk="Microsoft.NET.Sdk">
   <PropertyGroup>
-    <!--x-release-please-start-version-->
     <Version>1.1.0</Version>
-    <!--x-release-please-end-->
     <!-- The BUILDFRAMEWORKS variable allows us to override the target frameworks with a
          single framework that we are testing; this allows us to test with older SDK
          versions that would error out if they saw any newer target frameworks listed
diff --git a/release-please-config.json b/release-please-config.json
index 14c62590..e8b1982a 100644
--- a/release-please-config.json
+++ b/release-please-config.json
@@ -7,58 +7,100 @@
       "release-type": "simple",
       "package-name": "LaunchDarkly.ServerSdk.Consul",
       "extra-files": [
-        "src/LaunchDarkly.ServerSdk.Consul.csproj"
+        {
+          "type": "xml",
+          "path": "src/LaunchDarkly.ServerSdk.Consul.csproj",
+          "xpath": "//Project/PropertyGroup/Version"
+        }
       ]
     },
     "pkgs/dotnet-server-sdk-dynamodb": {
       "release-type": "simple",
       "package-name": "LaunchDarkly.ServerSdk.DynamoDb",
       "extra-files": [
-        "src/LaunchDarkly.ServerSdk.DynamoDb.csproj"
+        {
+          "type": "xml",
+          "path": "src/LaunchDarkly.ServerSdk.DynamoDb.csproj",
+          "xpath": "//Project/PropertyGroup/Version"
+        }
       ]
     },
     "pkgs/dotnet-server-sdk-redis": {
       "release-type": "simple",
       "package-name": "LaunchDarkly.ServerSdk.Redis",
       "extra-files": [
-        "src/LaunchDarkly.ServerSdk.Redis.csproj"
+        {
+          "type": "xml",
+          "path": "src/LaunchDarkly.ServerSdk.Redis.csproj",
+          "xpath": "//Project/PropertyGroup/Version"
+        }
       ]
     },
     "pkgs/sdk/server": {
+      "release-type": "simple",
       "package-name": "LaunchDarkly.ServerSdk",
       "extra-files": [
-        "src/LaunchDarkly.ServerSdk.csproj"
+        {
+          "type": "xml",
+          "path": "src/LaunchDarkly.ServerSdk.csproj",
+          "xpath": "//Project/PropertyGroup/Version"
+        }
       ]
     },
     "pkgs/sdk/server-ai": {
+      "release-type": "simple",
       "package-name": "LaunchDarkly.ServerSdk.Ai",
       "bump-minor-pre-major": true,
       "extra-files": [
-        "src/LaunchDarkly.ServerSdk.Ai.csproj"
+        {
+          "type": "xml",
+          "path": "src/LaunchDarkly.ServerSdk.Ai.csproj",
+          "xpath": "//Project/PropertyGroup/Version"
+        }
       ]
     },
     "pkgs/sdk/client": {
+      "release-type": "simple",
       "package-name": "LaunchDarkly.ClientSdk",
       "extra-files": [
-        "src/LaunchDarkly.ClientSdk.csproj"
+        {
+          "type": "xml",
+          "path": "src/LaunchDarkly.ClientSdk.csproj",
+          "xpath": "//Project/PropertyGroup/Version"
+        }
       ]
     },
     "pkgs/telemetry": {
+      "release-type": "simple",
       "package-name": "LaunchDarkly.ServerSdk.Telemetry",
       "extra-files": [
-        "src/LaunchDarkly.ServerSdk.Telemetry.csproj"
+        {
+          "type": "xml",
+          "path": "src/LaunchDarkly.ServerSdk.Telemetry.csproj",
+          "xpath": "//Project/PropertyGroup/Version"
+        }
       ]
     },
     "pkgs/shared/common": {
+      "release-type": "simple",
       "package-name": "LaunchDarkly.CommonSdk",
       "extra-files": [
-        "src/LaunchDarkly.CommonSdk.csproj"
+        {
+          "type": "xml",
+          "path": "src/LaunchDarkly.CommonSdk.csproj",
+          "xpath": "//Project/PropertyGroup/Version"
+        }
       ]
     },
     "pkgs/shared/common-json-net": {
+      "release-type": "simple",
       "package-name": "LaunchDarkly.CommonSdk.JsonNet",
       "extra-files": [
-        "src/LaunchDarkly.CommonSdk.JsonNet.csproj"
+        {
+          "type": "xml",
+          "path": "src/LaunchDarkly.CommonSdk.JsonNet.csproj",
+          "xpath": "//Project/PropertyGroup/Version"
+        }
       ]
     }
   }

From aab7e5d5b02a6ef0ffb18f8f548b5b5e2ed86c55 Mon Sep 17 00:00:00 2001
From: jsonbailey <jbailey@launchdarkly.com>
Date: Mon, 2 Jun 2025 13:33:22 +0000
Subject: [PATCH 2/3] Revert "chore: Identify release please version from
 config"

This reverts commit bad7fa7a99d0e97a06ff0989bc681dbf7027e57c.
---
 .../client/src/LaunchDarkly.ClientSdk.csproj  |  3 +
 .../src/LaunchDarkly.ServerSdk.Ai.csproj      |  2 +
 .../server/src/LaunchDarkly.ServerSdk.csproj  |  2 +
 .../LaunchDarkly.ServerSdk.Telemetry.csproj   |  2 +
 release-please-config.json                    | 60 +++----------------
 5 files changed, 18 insertions(+), 51 deletions(-)

diff --git a/pkgs/sdk/client/src/LaunchDarkly.ClientSdk.csproj b/pkgs/sdk/client/src/LaunchDarkly.ClientSdk.csproj
index abdedddf..3fd16c47 100644
--- a/pkgs/sdk/client/src/LaunchDarkly.ClientSdk.csproj
+++ b/pkgs/sdk/client/src/LaunchDarkly.ClientSdk.csproj
@@ -1,6 +1,9 @@
 <Project Sdk="Microsoft.NET.Sdk">
+
     <PropertyGroup>
+        <!--x-release-please-start-version-->
         <Version>5.4.0</Version>
+        <!--x-release-please-end-->
         <!-- The BUILDFRAMEWORKS variable allows us to override the target frameworks in
              a situation where we need to build only the .NET Standard target and do not
              want to even mention the Android/iOS/Mac/Windows targets, because we're in an environment
diff --git a/pkgs/sdk/server-ai/src/LaunchDarkly.ServerSdk.Ai.csproj b/pkgs/sdk/server-ai/src/LaunchDarkly.ServerSdk.Ai.csproj
index 3ed3edab..4ebabad3 100644
--- a/pkgs/sdk/server-ai/src/LaunchDarkly.ServerSdk.Ai.csproj
+++ b/pkgs/sdk/server-ai/src/LaunchDarkly.ServerSdk.Ai.csproj
@@ -1,6 +1,8 @@
 <Project Sdk="Microsoft.NET.Sdk">
   <PropertyGroup>
+    <!--x-release-please-start-version-->
     <Version>0.7.0</Version>
+    <!--x-release-please-end-->
     <!-- The BUILDFRAMEWORKS variable allows us to override the target frameworks with a
          single framework that we are testing; this allows us to test with older SDK
          versions that would error out if they saw any newer target frameworks listed
diff --git a/pkgs/sdk/server/src/LaunchDarkly.ServerSdk.csproj b/pkgs/sdk/server/src/LaunchDarkly.ServerSdk.csproj
index 4c4e49fd..42d55d47 100644
--- a/pkgs/sdk/server/src/LaunchDarkly.ServerSdk.csproj
+++ b/pkgs/sdk/server/src/LaunchDarkly.ServerSdk.csproj
@@ -1,6 +1,8 @@
 <Project Sdk="Microsoft.NET.Sdk">
   <PropertyGroup>
+    <!--x-release-please-start-version-->
     <Version>8.8.0</Version>
+    <!--x-release-please-end-->
     <!-- The BUILDFRAMEWORKS variable allows us to override the target frameworks with a
          single framework that we are testing; this allows us to test with older SDK
          versions that would error out if they saw any newer target frameworks listed
diff --git a/pkgs/telemetry/src/LaunchDarkly.ServerSdk.Telemetry.csproj b/pkgs/telemetry/src/LaunchDarkly.ServerSdk.Telemetry.csproj
index eb08bf60..0902ecec 100644
--- a/pkgs/telemetry/src/LaunchDarkly.ServerSdk.Telemetry.csproj
+++ b/pkgs/telemetry/src/LaunchDarkly.ServerSdk.Telemetry.csproj
@@ -1,6 +1,8 @@
 <Project Sdk="Microsoft.NET.Sdk">
   <PropertyGroup>
+    <!--x-release-please-start-version-->
     <Version>1.1.0</Version>
+    <!--x-release-please-end-->
     <!-- The BUILDFRAMEWORKS variable allows us to override the target frameworks with a
          single framework that we are testing; this allows us to test with older SDK
          versions that would error out if they saw any newer target frameworks listed
diff --git a/release-please-config.json b/release-please-config.json
index e8b1982a..14c62590 100644
--- a/release-please-config.json
+++ b/release-please-config.json
@@ -7,100 +7,58 @@
       "release-type": "simple",
       "package-name": "LaunchDarkly.ServerSdk.Consul",
       "extra-files": [
-        {
-          "type": "xml",
-          "path": "src/LaunchDarkly.ServerSdk.Consul.csproj",
-          "xpath": "//Project/PropertyGroup/Version"
-        }
+        "src/LaunchDarkly.ServerSdk.Consul.csproj"
       ]
     },
     "pkgs/dotnet-server-sdk-dynamodb": {
       "release-type": "simple",
       "package-name": "LaunchDarkly.ServerSdk.DynamoDb",
       "extra-files": [
-        {
-          "type": "xml",
-          "path": "src/LaunchDarkly.ServerSdk.DynamoDb.csproj",
-          "xpath": "//Project/PropertyGroup/Version"
-        }
+        "src/LaunchDarkly.ServerSdk.DynamoDb.csproj"
       ]
     },
     "pkgs/dotnet-server-sdk-redis": {
       "release-type": "simple",
       "package-name": "LaunchDarkly.ServerSdk.Redis",
       "extra-files": [
-        {
-          "type": "xml",
-          "path": "src/LaunchDarkly.ServerSdk.Redis.csproj",
-          "xpath": "//Project/PropertyGroup/Version"
-        }
+        "src/LaunchDarkly.ServerSdk.Redis.csproj"
       ]
     },
     "pkgs/sdk/server": {
-      "release-type": "simple",
       "package-name": "LaunchDarkly.ServerSdk",
       "extra-files": [
-        {
-          "type": "xml",
-          "path": "src/LaunchDarkly.ServerSdk.csproj",
-          "xpath": "//Project/PropertyGroup/Version"
-        }
+        "src/LaunchDarkly.ServerSdk.csproj"
       ]
     },
     "pkgs/sdk/server-ai": {
-      "release-type": "simple",
       "package-name": "LaunchDarkly.ServerSdk.Ai",
       "bump-minor-pre-major": true,
       "extra-files": [
-        {
-          "type": "xml",
-          "path": "src/LaunchDarkly.ServerSdk.Ai.csproj",
-          "xpath": "//Project/PropertyGroup/Version"
-        }
+        "src/LaunchDarkly.ServerSdk.Ai.csproj"
       ]
     },
     "pkgs/sdk/client": {
-      "release-type": "simple",
       "package-name": "LaunchDarkly.ClientSdk",
       "extra-files": [
-        {
-          "type": "xml",
-          "path": "src/LaunchDarkly.ClientSdk.csproj",
-          "xpath": "//Project/PropertyGroup/Version"
-        }
+        "src/LaunchDarkly.ClientSdk.csproj"
       ]
     },
     "pkgs/telemetry": {
-      "release-type": "simple",
       "package-name": "LaunchDarkly.ServerSdk.Telemetry",
       "extra-files": [
-        {
-          "type": "xml",
-          "path": "src/LaunchDarkly.ServerSdk.Telemetry.csproj",
-          "xpath": "//Project/PropertyGroup/Version"
-        }
+        "src/LaunchDarkly.ServerSdk.Telemetry.csproj"
       ]
     },
     "pkgs/shared/common": {
-      "release-type": "simple",
       "package-name": "LaunchDarkly.CommonSdk",
       "extra-files": [
-        {
-          "type": "xml",
-          "path": "src/LaunchDarkly.CommonSdk.csproj",
-          "xpath": "//Project/PropertyGroup/Version"
-        }
+        "src/LaunchDarkly.CommonSdk.csproj"
       ]
     },
     "pkgs/shared/common-json-net": {
-      "release-type": "simple",
       "package-name": "LaunchDarkly.CommonSdk.JsonNet",
       "extra-files": [
-        {
-          "type": "xml",
-          "path": "src/LaunchDarkly.CommonSdk.JsonNet.csproj",
-          "xpath": "//Project/PropertyGroup/Version"
-        }
+        "src/LaunchDarkly.CommonSdk.JsonNet.csproj"
       ]
     }
   }

From fdfa3bc942717363b37e1e558188da9cfed50d8e Mon Sep 17 00:00:00 2001
From: jsonbailey <jbailey@launchdarkly.com>
Date: Mon, 2 Jun 2025 13:37:12 +0000
Subject: [PATCH 3/3] decorate the version in the csproj file

After some tought, I prefer decorating the csproj file vs using xml xpath in the release-please configuration because it makes it clear in the project file that the version is being managed by release-please.
---
 .../src/LaunchDarkly.ServerSdk.Consul.csproj                    | 2 ++
 .../src/LaunchDarkly.ServerSdk.DynamoDB.csproj                  | 2 ++
 .../src/LaunchDarkly.ServerSdk.Redis.csproj                     | 2 ++
 .../common-json-net/src/LaunchDarkly.CommonSdk.JsonNet.csproj   | 2 ++
 pkgs/shared/common/src/LaunchDarkly.CommonSdk.csproj            | 2 ++
 5 files changed, 10 insertions(+)

diff --git a/pkgs/dotnet-server-sdk-consul/src/LaunchDarkly.ServerSdk.Consul.csproj b/pkgs/dotnet-server-sdk-consul/src/LaunchDarkly.ServerSdk.Consul.csproj
index fa42e5aa..b0180227 100644
--- a/pkgs/dotnet-server-sdk-consul/src/LaunchDarkly.ServerSdk.Consul.csproj
+++ b/pkgs/dotnet-server-sdk-consul/src/LaunchDarkly.ServerSdk.Consul.csproj
@@ -1,6 +1,8 @@
 <Project Sdk="Microsoft.NET.Sdk">
   <PropertyGroup>
+    <!--x-release-please-start-version-->
     <Version>5.0.0</Version>
+    <!--x-release-please-end-->
     <TargetFrameworks>netstandard2.0;net462</TargetFrameworks>
     <PackageId>LaunchDarkly.ServerSdk.Consul</PackageId>
     <AssemblyName>LaunchDarkly.ServerSdk.Consul</AssemblyName>
diff --git a/pkgs/dotnet-server-sdk-dynamodb/src/LaunchDarkly.ServerSdk.DynamoDB.csproj b/pkgs/dotnet-server-sdk-dynamodb/src/LaunchDarkly.ServerSdk.DynamoDB.csproj
index b8bad02d..45797225 100644
--- a/pkgs/dotnet-server-sdk-dynamodb/src/LaunchDarkly.ServerSdk.DynamoDB.csproj
+++ b/pkgs/dotnet-server-sdk-dynamodb/src/LaunchDarkly.ServerSdk.DynamoDB.csproj
@@ -1,6 +1,8 @@
 <Project Sdk="Microsoft.NET.Sdk">
   <PropertyGroup>
+    <!--x-release-please-start-version-->
     <Version>4.0.0</Version>
+    <!--x-release-please-end-->
     <TargetFrameworks>netstandard2.0;net462</TargetFrameworks>
     <PackageId>LaunchDarkly.ServerSdk.DynamoDB</PackageId>
     <AssemblyName>LaunchDarkly.ServerSdk.DynamoDB</AssemblyName>
diff --git a/pkgs/dotnet-server-sdk-redis/src/LaunchDarkly.ServerSdk.Redis.csproj b/pkgs/dotnet-server-sdk-redis/src/LaunchDarkly.ServerSdk.Redis.csproj
index 7bca6653..9162601c 100644
--- a/pkgs/dotnet-server-sdk-redis/src/LaunchDarkly.ServerSdk.Redis.csproj
+++ b/pkgs/dotnet-server-sdk-redis/src/LaunchDarkly.ServerSdk.Redis.csproj
@@ -1,6 +1,8 @@
 <Project Sdk="Microsoft.NET.Sdk">
   <PropertyGroup>
+    <!--x-release-please-start-version-->
     <Version>5.0.0</Version>
+    <!--x-release-please-end-->
     <TargetFrameworks>netstandard2.0;net462</TargetFrameworks>
     <AssemblyName>LaunchDarkly.ServerSdk.Redis</AssemblyName>
     <OutputType>Library</OutputType>
diff --git a/pkgs/shared/common-json-net/src/LaunchDarkly.CommonSdk.JsonNet.csproj b/pkgs/shared/common-json-net/src/LaunchDarkly.CommonSdk.JsonNet.csproj
index c9204ad5..bcfa310b 100644
--- a/pkgs/shared/common-json-net/src/LaunchDarkly.CommonSdk.JsonNet.csproj
+++ b/pkgs/shared/common-json-net/src/LaunchDarkly.CommonSdk.JsonNet.csproj
@@ -1,6 +1,8 @@
 <Project Sdk="Microsoft.NET.Sdk">
   <PropertyGroup>
+    <!--x-release-please-start-version-->
     <Version>7.0.0</Version>
+    <!--x-release-please-end-->
     <!-- The BUILDFRAMEWORKS variable allows us to override the target frameworks with a
      single framework that we are testing; this allows us to test with older SDK
      versions that would error out if they saw any newer target frameworks listed
diff --git a/pkgs/shared/common/src/LaunchDarkly.CommonSdk.csproj b/pkgs/shared/common/src/LaunchDarkly.CommonSdk.csproj
index 1351c972..60006e3c 100644
--- a/pkgs/shared/common/src/LaunchDarkly.CommonSdk.csproj
+++ b/pkgs/shared/common/src/LaunchDarkly.CommonSdk.csproj
@@ -1,6 +1,8 @@
 <Project Sdk="Microsoft.NET.Sdk">
   <PropertyGroup>
+    <!--x-release-please-start-version-->
     <Version>7.0.0</Version>
+    <!--x-release-please-end-->
     <!-- The BUILDFRAMEWORKS variable allows us to override the target frameworks with a
      single framework that we are testing; this allows us to test with older SDK
      versions that would error out if they saw any newer target frameworks listed