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

prevent js-yaml.dump from line-wrapping strings #51

Merged

pauloancheta merged 1 commit into master from prevent-string-linewrapping

Jan 23, 2018

Collaborator

tavisrudd commented Jan 21, 2018

by setting the lineWidth to 999.

js-yaml was converting strings with lines longer than 80 columns into the > yaml string encoding, whereas we prefer | with no line-wrapping.


          prevent js-yaml.dump from line-wrapping strings

470eaa4

by setting the lineWidth to 999.

tavisrudd requested a review from pauloancheta

January 21, 2018 00:52

pauloancheta approved these changes

View reviewed changes

Contributor

pauloancheta left a comment

LGTM 👍

pauloancheta merged commit e6000f9 into master

pauloancheta deleted the prevent-string-linewrapping branch

January 23, 2018 01:11

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

Labels

None yet