Skip to content

Conversation

@emh-jump
Copy link
Contributor

@emh-jump emh-jump commented Aug 7, 2025

After #911, there is a regression for files which have only a single document separator (ie my added test case passed before that PR, failed once it was merged, and is fixed by this change)

cc @cppforlife another one for you to review/merge/release when you return, thank you!

@carvel-bot carvel-bot added this to Carvel Aug 7, 2025
@emh-jump emh-jump changed the title Fix ytt fmt with a single document separator Fix ytt fmt for documents with a single document separator Aug 7, 2025
@emh-jump emh-jump force-pushed the empty-doc-seperator branch from 03e0263 to 12d242a Compare August 7, 2025 21:55
Signed-off-by: Ethan Hampton <[email protected]>
@emh-jump emh-jump force-pushed the empty-doc-seperator branch from 12d242a to e816fb7 Compare August 7, 2025 21:58
@emh-jump emh-jump changed the title Fix ytt fmt for documents with a single document separator Fix ytt fmt for files with a single document separator Aug 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

1 participant