-
Notifications
You must be signed in to change notification settings - Fork 654
GH2139: GitVersion logging does not include newlines #2146
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.
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
Conversation
Linux raw logs from GitHub Actions captured before this change:
Linux raw logs from GitHub Actions captured after this change:
Note that after the change I also forced a failure using As shown, the logging change had the desired effect and the GitVersion log is now much easier to read. PS. It took me a while to find the AppVeyor NuGet feed at https://ci.appveyor.com/nuget/gitversion-8nigugxjftrw, needed to test with this PR. It should probably be documented somewhere? |
Does not improve log readability. Prefer to keep vertical space compact.
Check for "Build / Unit Test (macos-latest) (pull_request)" seems to have had an unrelated failure. |
Re-run the build, now it's fine |
Mac raw logs from GitHub Actions captured after this change:
Windows raw logs from GitHub Actions captured after this change:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Thanks for your contributions, @gitfool! 🙏 |
Fixes #2139.
I also tried to fix newlines to be platform agnostic.
Finally, I suppressed CS0618 and NU1701 build warnings.