Skip to content

switch over from serde-yaml to serde-saphyr#82

Merged
clux merged 6 commits into
mainfrom
saphyr
Mar 10, 2026
Merged

switch over from serde-yaml to serde-saphyr#82
clux merged 6 commits into
mainfrom
saphyr

Conversation

@clux

@clux clux commented Feb 23, 2026

Copy link
Copy Markdown
Owner

mostly non-breaking (2 tiny stylistic output things in tests).

notes (most got resolved during development)

  1. list output seems to force more --- document separators, but it seems pretty sensible
  2. compact list output seems missing - raised in support serializing with compact indentation of lists bourumir-wyngs/serde-saphyr#86
  3. multidoc with options seems slightly awkward since no helper fn for it - asked in support options for to_string_multiple bourumir-wyngs/serde-saphyr#87
  4. struggling to use readers with multiple and options, maybe i have to inline from_multiple_with_options for a from_reader_with_options or maybe they need a combinatorial from_reader_multiple_with_options..

1 is no problem really. 4. can be worked around at the cost of some internal buffering, asking upstream about 2 + 3.

otherwise it's nicer in many ways. not having to deal with singleton_map_recursive is nice.

clux added 4 commits February 23, 2026 16:59
Signed-off-by: clux <sszynrae@gmail.com>
Signed-off-by: clux <sszynrae@gmail.com>
Signed-off-by: clux <sszynrae@gmail.com>
Signed-off-by: clux <sszynrae@gmail.com>
clux added 2 commits March 10, 2026 20:11
Signed-off-by: clux <sszynrae@gmail.com>
Signed-off-by: clux <sszynrae@gmail.com>
@clux clux marked this pull request as ready for review March 10, 2026 20:36
@clux clux merged commit d0a9c00 into main Mar 10, 2026
14 checks passed
@clux clux deleted the saphyr branch March 10, 2026 20:36
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