Skip to content

Commit

Permalink
Apply suggestions from code review (/nr: => /nodeReuse:)
Browse files Browse the repository at this point in the history
  • Loading branch information
augustoproiete committed Jul 16, 2023
1 parent a53db97 commit 449eb23
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -814,7 +814,7 @@ public void Should_Use_Node_Reuse_If_Specified()
var result = fixture.Run();

// Then
Assert.Equal("msbuild /nr:true", result.Args);
Assert.Equal("msbuild /nodeReuse:true", result.Args);
}
}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -204,9 +204,9 @@ public static void AppendMSBuildSettings(this ProcessArgumentBuilder builder, Do
}

// Re-use of MSBuild nodes?
if (settings.NodeReuse != null)
if (settings.NodeReuse.HasValue)
{
msBuilder.Append(string.Concat("/nr:", settings.NodeReuse.Value ? "true" : "false"));
msBuilder.Append(string.Concat("/nodeReuse:", settings.NodeReuse.Value ? "true" : "false"));
}

builder.AppendRange(
Expand Down

0 comments on commit 449eb23

Please sign in to comment.