Skip to content

Commit

Permalink
use Environment.NewLine
Browse files Browse the repository at this point in the history
  • Loading branch information
oleksandr-didyk committed May 31, 2023
1 parent 23a67c4 commit 0407496
Showing 1 changed file with 6 additions and 5 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@
using Microsoft.Build.Framework;
using Microsoft.Build.Utilities;
using NuGet.Packaging.Core;
using System;
using System.Collections.Generic;
using System.IO;
using System.Linq;
Expand Down Expand Up @@ -42,12 +43,12 @@ public class ValidateUsageAgainstBaseline : Microsoft.Build.Utilities.Task

private readonly string _reviewRequestMessage = "Whenever altering this " +
"or other Source Build files, please make sure to include @dotnet/source-build-internal " +
"as a reviewer";
"as a reviewer.";

public override bool Execute()
{
string ReviewRequestComment = $"<!-- {_reviewRequestMessage} -->\n";
string PreBuiltDocXmlComment = $"<!-- {_preBuiltDocMessage} -->\n";
string ReviewRequestComment = $"<!-- {_reviewRequestMessage} -->{Environment.NewLine}";
string PreBuiltDocXmlComment = $"<!-- {_preBuiltDocMessage} -->{Environment.NewLine}";

var used = UsageData.Parse(XElement.Parse(File.ReadAllText(DataFile)));

Expand Down Expand Up @@ -89,8 +90,8 @@ public UsageValidationData GetUsageValidationData(UsageData baseline, UsageData
tellUserToUpdateBaseline = true;
Log.LogError(
$"{diff.Added.Length} new pre-builts discovered! Detailed usage " +
$"report can be found at {OutputReportFile}.\n{_preBuiltDocMessage}\n" +
$"Package IDs are:\n" + string.Join("\n", diff.Added.Select(u => u.ToString())));
$"report can be found at {OutputReportFile}.{Environment.NewLine}{_preBuiltDocMessage}{Environment.NewLine}" +
$"Package IDs are:{Environment.NewLine}" + string.Join(Environment.NewLine, diff.Added.Select(u => u.ToString())));

// In the report, list full usage info, not only identity.
report.Add(
Expand Down

0 comments on commit 0407496

Please sign in to comment.