From 821aef07d63839566868ce4284e1570e1e6d4a62 Mon Sep 17 00:00:00 2001
From: Ismo Vuorinen <ismo@ivuorinen.net>
Date: Fri, 21 Nov 2025 08:22:54 +0200
Subject: [PATCH] fix: remove misleading 'Restore Complete' step in
 csharp-publish

Remove the 'Restore Complete' step that always printed 'Cache hit -
skipping dotnet restore' even though restore always runs via the retry
action.

The message was misleading because:
- Dependencies are always restored via step-security/retry
- The message claimed restore was skipped, which was false
- The step served no actual purpose

The 'Restore Dependencies' step already provides appropriate output
during execution, making this step redundant and confusing.
---
 csharp-publish/action.yml | 5 -----
 1 file changed, 5 deletions(-)

diff --git a/csharp-publish/action.yml b/csharp-publish/action.yml
index 11106bb..abbb0d6 100644
--- a/csharp-publish/action.yml
+++ b/csharp-publish/action.yml
@@ -177,11 +177,6 @@ runs:
           echo "Restoring .NET dependencies..."
           dotnet restore --verbosity normal
 
-    - name: Restore Complete
-      shell: bash
-      run: |
-        echo "Cache hit - skipping dotnet restore"
-
     - name: Build Solution
       shell: bash
       run: |