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.

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

Fix tag encode in mapping #401

Closed
wants to merge 2 commits into from
Closed

Fix tag encode in mapping #401

wants to merge 2 commits into from

Conversation

RangelReale
Copy link

When encoding a yaml tag in a map key, the tag appears on the next line.

got:

        a:
          b:
            c:
        !!timestamp test

expected:

        a:
          b:
            c: !!timestamp test

This PR checks for TagNode and encode without the newline.

@goccy goccy mentioned this pull request Nov 13, 2024
Open
@LarsStegman
Copy link
Contributor

@RangelReale any chance of making it possible to merge this? Are you open to me branching off your branch and fixing it?

@RangelReale
Copy link
Author

RangelReale commented Jan 31, 2025 via email

@LarsStegman LarsStegman mentioned this pull request Jan 31, 2025
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
encoder high priority
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@RangelReale @LarsStegman @goccy