Skip to content

added encoding to kyaml-compact and kyaml-pretty#64

Merged
zkry merged 1 commit intozkry:masterfrom
ehrt74:master
Jan 11, 2026
Merged

added encoding to kyaml-compact and kyaml-pretty#64
zkry merged 1 commit intozkry:masterfrom
ehrt74:master

Conversation

@ehrt74
Copy link
Contributor

@ehrt74 ehrt74 commented Dec 20, 2025

@zkry
Copy link
Owner

zkry commented Jan 11, 2026

Thanks for this contribution! I haven't heard of kyaml, I think it's a good idea to have more encoding options.

@ehrt74
Copy link
Contributor Author

ehrt74 commented Jan 11, 2026

i think it's a really nice subset of yaml. it's not whitespace sensitive, so you no longer need to litter helm charts with "indent". prot uses it (sort of) for meta-information in the denote package and i'd like to send him a pull request where i use yaml.el to parse and create meta-information in denote.

@zkry
Copy link
Owner

zkry commented Jan 11, 2026

This looks good for now. I may end up overhauling the encoding with regards to the flow output (that kyaml wants). One thing in particular is to have a column width, so like it joins them on the same line as long as the line stays below a set width. I'll post any updates regarding this here. When things are finalized I'll do a release and then you should be good to use this feature in other packages.

@zkry zkry merged commit 920f71b into zkry:master Jan 11, 2026
1 check failed
@zkry
Copy link
Owner

zkry commented Jan 12, 2026

@ehrt74 Ok, release is out now so you should be good to use it. I just changed the names of the two variables you added to yaml-encode-indent-width and yaml-encode-dialect, as well as ensuring that the output doesn't have trailing commas.

@ehrt74
Copy link
Contributor Author

ehrt74 commented Jan 12, 2026

oh, the output should have trailing commas. one of the ideas of kyaml is that you should be able to change the order of elements in a list or a map without worrying about adding or removing commas

@zkry
Copy link
Owner

zkry commented Jan 12, 2026

oh ok, I'll update this then to leave the trailing commas. Here's the PR: #65

@zkry
Copy link
Owner

zkry commented Jan 13, 2026

Ok, should be good now with a new release (v 1.2.3)👍

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