Skip to content

Commit

Permalink
Merge pull request #664 from JanKrivanek/proto/nodes-management-node
Browse files Browse the repository at this point in the history
Add Nodes Management node for node reusal messages
  • Loading branch information
KirillOsenkov authored Mar 17, 2023
2 parents df31e06 + 470bde0 commit 3b0eeba
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 0 deletions.
4 changes: 4 additions & 0 deletions src/StructuredLogger/Construction/MessageProcessor.cs
Original file line number Diff line number Diff line change
Expand Up @@ -502,6 +502,10 @@ public void AddMessage(LazyFormattedBuildEventArgs args, string message)
return;
}
}
else if (args.Message.StartsWith(Strings.NodesReusal, StringComparison.Ordinal))
{
parent = construction.Build.GetOrCreateNodeWithName<Folder>(Strings.NodesManagementNode);
}

if (parent == null)
{
Expand Down
2 changes: 2 additions & 0 deletions src/StructuredLogger/Strings/Strings.cs
Original file line number Diff line number Diff line change
Expand Up @@ -507,6 +507,8 @@ public static Match IsFoundConflicts(string text)
public static string MSBuildVersionPrefix => "MSBuild version = ";
public static string MSBuildExecutablePathPrefix => "MSBuild executable path = ";
public static string Warnings = "Warnings";
public static string NodesReusal = "Reusing node";
public static string NodesManagementNode = "Nodes Management";

// These aren't localized, see https://github.com/microsoft/MSBuildSdks/blob/543e965191417dee65471ee57a6702289847b49b/src/Artifacts/Tasks/Robocopy.cs#L66-L77
private const string RobocopyFileCopiedMessage = "Copied {0} to {1}";
Expand Down

0 comments on commit 3b0eeba

Please sign in to comment.