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.

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

fix: set yaml line width to infinity #1070

Closed
wants to merge 2 commits into from

Conversation

MatteoVoges
Copy link
Contributor

@MatteoVoges MatteoVoges commented Oct 6, 2023

Proposed Changes

  • sets the yaml dump width to inf
  • this fixes an issue, that yaml uses 80 characters by default until an automatic linebreak gets inserted. For long expressions this is not suitable.

IMPORTANT: This may be a breaking change, but no one should really depend on automatic line breaks at the limit of 80 characters.
I would offer, to make it configurable via a flag, if it is necessary.

stackoverflow-solution for reference

@MatteoVoges MatteoVoges added the bugfix fixing a bug label Oct 6, 2023
@MatteoVoges MatteoVoges self-assigned this Oct 6, 2023
@MatteoVoges
Copy link
Contributor Author

Like mentioned in #1080 this will get controllable and won't be a breaking change anymore

@MatteoVoges MatteoVoges deleted the fix/yaml-linebreaks branch October 30, 2023 17:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bugfix fixing a bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant