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

Add a :compact option to strip whitespace off terminal nodes in pretty.rb #150

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

shreeve
Copy link

@shreeve shreeve commented Sep 17, 2024

I didn't see a way to do this, so I am submitting a PR to add the ability to strip whitespace off terminal nodes when pretty printing.

For example, instead of this:

foo Hello
bar World
baz
  bat Wow!

generating this:

<foo>
  Hello
</foo>
<bar>
  World
</bar>
<baz>
  <bat>
    Wow!
  </bat>
</baz>

The compact: true option would generate:

<foo>Hello</foo>
<bar>World</bar>
<baz>
  <bat>Wow!</bat>
</baz>

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.

1 participant