From 8a761c8097ce3767eaa1dacced3fc8fdc38b657b Mon Sep 17 00:00:00 2001 From: Genevieve Warren <24882762+gewarren@users.noreply.github.com> Date: Fri, 11 Sep 2026 17:49:58 -0700 Subject: [PATCH 1/3] Log file name up front --- .../src/RedirectionVerifier/RedirectionHelpers.cs | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/actions/docs-verifier/src/RedirectionVerifier/RedirectionHelpers.cs b/actions/docs-verifier/src/RedirectionVerifier/RedirectionHelpers.cs index b8dd2e44..0a2c9921 100644 --- a/actions/docs-verifier/src/RedirectionVerifier/RedirectionHelpers.cs +++ b/actions/docs-verifier/src/RedirectionVerifier/RedirectionHelpers.cs @@ -15,11 +15,7 @@ public static async Task> GetRedirectionFileNames() // If no redirection files are found in the OPS config, just use the default name. ImmutableArray redirectionFileNames = await GetRedirectionFilesAsync() ?? [".openpublishing.redirection.json"]; - Console.WriteLine($"The following {redirectionFileNames.Length} redirection files are registered:"); - foreach (string filename in redirectionFileNames) - { - Console.WriteLine(filename); - } + Console.WriteLine($"Found {redirectionFileNames.Length} registered redirection files."); return redirectionFileNames; } From f432902f274233fa18d14de04aa9e36156fc2c85 Mon Sep 17 00:00:00 2001 From: Genevieve Warren <24882762+gewarren@users.noreply.github.com> Date: Fri, 11 Sep 2026 17:50:58 -0700 Subject: [PATCH 2/3] Log file name up front --- .../src/RedirectionVerifier/RedirectTargetVerifier.cs | 2 ++ 1 file changed, 2 insertions(+) diff --git a/actions/docs-verifier/src/RedirectionVerifier/RedirectTargetVerifier.cs b/actions/docs-verifier/src/RedirectionVerifier/RedirectTargetVerifier.cs index 6a3817c3..a1782002 100644 --- a/actions/docs-verifier/src/RedirectionVerifier/RedirectTargetVerifier.cs +++ b/actions/docs-verifier/src/RedirectionVerifier/RedirectTargetVerifier.cs @@ -39,6 +39,8 @@ await WriteErrorAsync( return false; } + await writer.WriteLineAsync($"Checking redirect targets in '{redirectionFilePath}'..."); + OpenPublishingRedirectionReader reader = new(redirectionFilePath); ImmutableArray redirections = await reader.MapConfigurationAsync(); if (redirections.IsDefaultOrEmpty) From 874b3c78bdb2ce7280d10ef43ddee18ec16915e6 Mon Sep 17 00:00:00 2001 From: Genevieve Warren <24882762+gewarren@users.noreply.github.com> Date: Fri, 11 Sep 2026 17:55:23 -0700 Subject: [PATCH 3/3] fix tests --- .../tests/GitHub.UnitTests/RedirectTargetVerifierTests.cs | 2 -- 1 file changed, 2 deletions(-) diff --git a/actions/docs-verifier/tests/GitHub.UnitTests/RedirectTargetVerifierTests.cs b/actions/docs-verifier/tests/GitHub.UnitTests/RedirectTargetVerifierTests.cs index 3f2ecd97..bc1ae9d5 100644 --- a/actions/docs-verifier/tests/GitHub.UnitTests/RedirectTargetVerifierTests.cs +++ b/actions/docs-verifier/tests/GitHub.UnitTests/RedirectTargetVerifierTests.cs @@ -19,7 +19,6 @@ public async Task WriteResultsAsyncReturnsTrueForValidLearnUrlPath() _ => Task.FromResult(HttpStatusCode.OK)); Assert.True(result); - Assert.Equal(string.Empty, writer.ToString()); } finally { @@ -46,7 +45,6 @@ public async Task WriteResultsAsyncSkipsNonLearnUrlTargets() }); Assert.True(result); - Assert.Equal(string.Empty, writer.ToString()); Assert.False(statusProviderCalled); } finally