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

Enable the build logger fix for CR and LF by default #3310

Merged
merged 1 commit into from
May 17, 2021
Merged

Enable the build logger fix for CR and LF by default #3310

merged 1 commit into from
May 17, 2021

Conversation

steakhal
Copy link
Contributor

No description provided.

@steakhal
Copy link
Contributor Author

The diff looks ugly, but the change is simple.

@steakhal steakhal marked this pull request as ready for review May 13, 2021 12:02
@steakhal steakhal requested a review from bruntib as a code owner May 13, 2021 12:02
@csordasmarton csordasmarton added this to the release 6.16.0 milestone May 17, 2021
csordasmarton
csordasmarton approved these changes May 17, 2021
View reviewed changes
Copy link
Contributor

@csordasmarton csordasmarton left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@csordasmarton csordasmarton merged commit a682daf into Ericsson:master May 17, 2021
@steakhal steakhal deleted the buildlogger-enable-new-escaping branch January 17, 2022 12:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@csordasmarton csordasmarton csordasmarton approved these changes

@bruntib bruntib Awaiting requested review from bruntib bruntib is a code owner

Assignees
No one assigned
Labels
ld-logger 📃 refactoring 😡 ➡️ 🙂 Refactoring code.
Projects
None yet
Milestone
release 6.16.0
Development

Successfully merging this pull request may close these issues.
2 participants
@steakhal @csordasmarton