Merge pull request godotengine#103714 from raulsntos/dotnet/skip-resaving-when-tfm-unchanged

Repiteo · Repiteo · commit f002258454ee · 2025-03-06T16:36:08.000-06:00
[.NET] Skip re-saving `.csproj` when TFM is unchanged
diff --git a/modules/mono/editor/GodotTools/GodotTools.ProjectEditor/ProjectUtils.cs b/modules/mono/editor/GodotTools/GodotTools.ProjectEditor/ProjectUtils.cs
@@ -191,6 +191,13 @@ private static void EnsureTargetFrameworkMatchesMinimumRequirement(MSBuildProjec
                 // Otherwise, it can be removed.
                 if (mainTfmVersion > minTfmVersion)
                 {
+                    var propertyTfmVersion = NuGetFramework.Parse(property.Value).Version;
+                    if (propertyTfmVersion == minTfmVersion)
+                    {
+                        // The 'TargetFramework' property already matches the minimum version.
+                        continue;
+                    }
+
                     property.Value = minTfmValue;
                 }
                 else

Original file line number	Diff line number	Diff line change
`@@ -191,6 +191,13 @@ private static void EnsureTargetFrameworkMatchesMinimumRequirement(MSBuildProjec`
`191`	`191`	`// Otherwise, it can be removed.`
`192`	`192`	`if (mainTfmVersion > minTfmVersion)`
`193`	`193`	`{`
	`194`	`+ var propertyTfmVersion = NuGetFramework.Parse(property.Value).Version;`
	`195`	`+ if (propertyTfmVersion == minTfmVersion)`
	`196`	`+ {`
	`197`	`+ // The 'TargetFramework' property already matches the minimum version.`
	`198`	`+ continue;`
	`199`	`+ }`
	`200`	`+`
`194`	`201`	`property.Value = minTfmValue;`
`195`	`202`	`}`
`196`	`203`	`else`