Skip to content

Add option to specify max line width.#223

Merged
puzrin merged 1 commit intonodeca:masterfrom
APIs-guru:master
Nov 23, 2015
Merged

Add option to specify max line width.#223
puzrin merged 1 commit intonodeca:masterfrom
APIs-guru:master

Conversation

@IvanGoncharov
Copy link
Copy Markdown
Contributor

No description provided.

@puzrin
Copy link
Copy Markdown
Member

puzrin commented Nov 23, 2015

Just one question. Is it really necessary to tune width?

@IvanGoncharov
Copy link
Copy Markdown
Contributor Author

In my case yes, I do some transformations on handwritten YAML files and compare them afterwards.
I work with API descriptions and it most of the cases they have quite long descriptions inside YAML.
For example: https://github.com/darklynx/swagger-api-collection/blob/master/api/yaml/instagram.yaml#L18
So my intent is to set this values as -1.

I took semantic from here: http://stackoverflow.com/questions/17859864/why-does-psych-yaml-interpreter-add-line-breaks-around-80-characters
So I think it useful not only to me 😄

puzrin pushed a commit that referenced this pull request Nov 23, 2015
Add option to specify max line width.
@puzrin puzrin merged commit 73419ba into nodeca:master Nov 23, 2015
@IvanGoncharov
Copy link
Copy Markdown
Contributor Author

Wow it was super fast 👍

@puzrin
Copy link
Copy Markdown
Member

puzrin commented Nov 23, 2015

Ok. If you dig closed tickets, you can see that i postponed this feature until someone say "yes i need it" :)

@IvanGoncharov
Copy link
Copy Markdown
Contributor Author

I found it: #215 (comment)
@puzrin Can you please make release?
This change is blocking me + it allow me to workaround #222

@puzrin
Copy link
Copy Markdown
Member

puzrin commented Nov 23, 2015

done

@IvanGoncharov
Copy link
Copy Markdown
Contributor Author

Thanks a lot

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants