Fix returning a path

dotnet · Nov 26, 2024 · 1f528e4 · 1f528e4
1 parent ec30ea8
commit 1f528e4
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/...roductConstructionService/ProductConstructionService.DependencyFlow/PullRequestUpdater.cs b/...roductConstructionService/ProductConstructionService.DependencyFlow/PullRequestUpdater.cs
@@ -1107,12 +1107,12 @@ private async Task<bool> UpdateAssetsAndSources(SubscriptionUpdateWorkItem updat
         {
             if (isForwardFlow)
             {
-                targetRepo = _vmrInfo.VmrPath;
                 hadUpdates = await _vmrForwardFlower.FlowForwardAsync(
                     subscription,
                     build,
                     newBranchName,
                     cancellationToken: default);
+                targetRepo = _vmrInfo.VmrPath;
             }
             else
             {
@@ -1140,7 +1140,7 @@ private async Task<bool> UpdateAssetsAndSources(SubscriptionUpdateWorkItem updat
 
         _logger.LogInformation("Code changes for {subscriptionId} ready in local branch {branch}",
             subscription.Id,
-            subscription.TargetBranch);
+            newBranchName);
 
         // TODO https://github.com/dotnet/arcade-services/issues/3318: Handle failures (conflict, non-ff etc)
         using (var scope = _telemetryRecorder.RecordGitOperation(TrackedGitOperation.Push, subscription.TargetRepository))