8000
Skip to content

Make line endings configurable#424

Merged
akshayjshah merged 2 commits intouber-go:masterfrom
alaczi:line_ending
May 5, 2017
Merged

Make line endings configurable#424
akshayjshah merged 2 commits intouber-go:masterfrom
alaczi:line_ending

Conversation

@alaczi
Copy link
Copy Markdown
Contributor
@alaczi alaczi commented May 4, 2017

Add LineEnding to EncoderConfig, which allows Windows users to use non-Unix newlines.

Fixes #412.

@CLAassistant
Copy link
Copy Markdown
CLAassistant commented May 4, 2017

CLA assistant check
All committers have signed the CLA.

…g LineSeparator to the encoder configuration
@akshayjshah akshayjshah changed the title Adding possibility to configure line separator for log lines Make line endings configurable May 5, 2017
@akshayjshah
Copy link
Copy Markdown
Contributor

@alaczi This is great - thanks for the contribution! The test coverage is great.

I'm going to push a commit that changes LineSeparator to LineEnding, which is the term I see used more widely. Other than that small change, this is ready to merge 🎉

Copy link
Copy Markdown
Contributor
@akshayjshah akshayjshah left a comment

Choose a reason for hiding this comment

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

gr8

< 8000 circle cx="8" cy="8" r="7" stroke="currentColor" stroke-opacity="0.25" stroke-width="2" vector-effect="non-scaling-stroke" fill="none">

@akshayjshah akshayjshah merged commit cea6c27 into uber-go:master May 5, 2017
@alaczi alaczi deleted the line_ending branch May 8, 2017 08:09
@alaczi
Copy link
Copy Markdown
Contributor Author
alaczi commented May 8, 2017

Cheers!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

0