Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

GitHub actions file output has redundant entries #727

Merged
merged 2 commits into from Mar 6, 2022
Merged

GitHub actions file output has redundant entries #727

merged 2 commits into from Mar 6, 2022

Commits on Mar 6, 2022

  1. Break out of for loop on success 

    Addresses item 1 of issue dotnet#726
    @Hugo-Dahl-Tyler @AArnott
    Hugo-Dahl-Tyler authored and AArnott committed Mar 6, 2022
    Copy the full SHA
    4d9c33d View commit details
    Browse the repository at this point in the history

  2. Use 'File.AppendLine()' to generate output 

    Let the 'System.IO.File' provider handle newlines between entries rather than manually inserting them
Addresses item 2 of dotnet#726
    @Hugo-Dahl-Tyler @AArnott
    Hugo-Dahl-Tyler authored and AArnott committed Mar 6, 2022
    Copy the full SHA
    b6e44c3 View commit details
    Browse the repository at this point in the history