From 5e7d906d8870dbf783c1742722a67cdf0b484fc5 Mon Sep 17 00:00:00 2001
From: Logan Bussell <loganbussell@microsoft.com>
Date: Tue, 17 Dec 2024 11:50:16 -0800
Subject: [PATCH] Use private field for script path

---
 .../ScriptRunnerUpdater.cs                    | 25 ++++++++-----------
 1 file changed, 11 insertions(+), 14 deletions(-)

diff --git a/eng/update-dependencies/ScriptRunnerUpdater.cs b/eng/update-dependencies/ScriptRunnerUpdater.cs
index b9e69b5e86..2d761bce3c 100644
--- a/eng/update-dependencies/ScriptRunnerUpdater.cs
+++ b/eng/update-dependencies/ScriptRunnerUpdater.cs
@@ -15,26 +15,23 @@ namespace Dotnet.Docker
     /// </summary>
     public class ScriptRunnerUpdater : IDependencyUpdater
     {
-        public required string ScriptPath { get; init; }
+        private readonly string _scriptPath;
 
-        private ScriptRunnerUpdater()
+        private ScriptRunnerUpdater(string scriptPath)
         {
+            _scriptPath = scriptPath;
         }
 
         public static IDependencyUpdater GetDockerfileUpdater(string repoRoot)
         {
-            return new ScriptRunnerUpdater()
-            {
-                ScriptPath = Path.Combine(repoRoot, "eng", "dockerfile-templates", "Get-GeneratedDockerfiles.ps1")
-            };
+            string scriptPath = Path.Combine(repoRoot, "eng", "dockerfile-templates", "Get-GeneratedDockerfiles.ps1");
+            return new ScriptRunnerUpdater(scriptPath);
         }
 
         public static IDependencyUpdater GetReadMeUpdater(string repoRoot)
         {
-            return new ScriptRunnerUpdater()
-            {
-                ScriptPath = Path.Combine(repoRoot, "eng", "readme-templates", "Get-GeneratedReadmes.ps1")
-            };
+            string scriptPath = Path.Combine(repoRoot, "eng", "readme-templates", "Get-GeneratedReadmes.ps1");
+            return new ScriptRunnerUpdater(scriptPath);
         }
 
         public IEnumerable<DependencyUpdateTask> GetUpdateTasks(IEnumerable<IDependencyInfo> dependencyInfos) =>
@@ -47,24 +44,24 @@ public IEnumerable<DependencyUpdateTask> GetUpdateTasks(IEnumerable<IDependencyI
 
         private void ExecuteScript()
         {
-            Trace.TraceInformation($"Executing '{ScriptPath}'");
+            Trace.TraceInformation($"Executing '{_scriptPath}'");
 
             // Support both execution within Windows 10, Nano Server and Linux environments.
             Process process;
             try
             {
-                process = Process.Start("pwsh", ScriptPath);
+                process = Process.Start("pwsh", _scriptPath);
                 process.WaitForExit();
             }
             catch (Win32Exception)
             {
-                process = Process.Start("powershell", ScriptPath);
+                process = Process.Start("powershell", _scriptPath);
                 process.WaitForExit();
             }
 
             if (process.ExitCode != 0)
             {
-                throw new InvalidOperationException($"Unable to successfully execute '{ScriptPath}'");
+                throw new InvalidOperationException($"Unable to successfully execute '{_scriptPath}'");
             }
         }
     }