Merge pull request pyrevitlabs#3496 from pyrevitlabs/copilot/fix-failing-github-actions-job

jmcouffin · web-flow · commit 57caae267703 · 2026-07-21T23:35:41.000+02:00
Fix CI build break from uninitialized `isWritable` in `PyRevitConfigs`
diff --git a/dev/pyRevitLabs/pyRevitLabs.PyRevit/PyRevitConfigs.cs b/dev/pyRevitLabs/pyRevitLabs.PyRevit/PyRevitConfigs.cs
@@ -59,11 +59,12 @@ public static PyRevitConfig GetConfigFile()
             {
                 if (CommonUtils.VerifyFile(adminConfig))
                 {
-                    bool isWritable;
+                    bool isWritable = false;
                     try
                     {
 // Probe write access to detect ACL-protected files that are not marked with the ReadOnly attribute.
 using (var fs = new FileStream(adminConfig, FileMode.Open, FileAccess.Write, FileShare.ReadWrite)) { }
+                        isWritable = true;
                     }
                     catch (UnauthorizedAccessException)
                     {

Original file line number	Diff line number	Diff line change
`@@ -59,11 +59,12 @@ public static PyRevitConfig GetConfigFile()`
`59`	`59`	`{`
`60`	`60`	`if (CommonUtils.VerifyFile(adminConfig))`
`61`	`61`	`{`
`62`		`- bool isWritable;`
	`62`	`+ bool isWritable = false;`
`63`	`63`	`try`
`64`	`64`	`{`
`65`	`65`	`// Probe write access to detect ACL-protected files that are not marked with the ReadOnly attribute.`
`66`	`66`	`using (var fs = new FileStream(adminConfig, FileMode.Open, FileAccess.Write, FileShare.ReadWrite)) { }`
	`67`	`+ isWritable = true;`
`67`	`68`	`}`
`68`	`69`	`catch (UnauthorizedAccessException)`
`69`	`70`	`{`