Skip to content

Update textwrap to 0.16.4 and simplify indentation calls - #101

Open
mgeisler wants to merge 1 commit into
zefchain:mainfrom
mgeisler:update-textwrap
Open

mgeisler wants to merge 1 commit into
zefchain:mainfrom
mgeisler:update-textwrap

Conversation

@mgeisler

@mgeisler mgeisler commented Sep 15, 2026 •

Copy link
Copy Markdown

Summary

Update textwrap dependency in serde-generate from 0.13.4 to 0.16.4.

Since textwrap::indent automatically prefixes empty lines with the trimmed prefix (e.g. //, ///, *), the post-processing .replace calls across all code generators (cpp, csharp, dart, golang, java, kotlin, rust, swift, typescript) are redundant and have been removed.

Test Plan

I ran cargo test for the Rust and Go generators.

@mgeisler
mgeisler requested a review from ma2bd as a code owner September 15, 2026 19:48
…ntation calls

Update `textwrap` dependency in `serde-generate` from 0.13.4 to 0.16.4.

Since `textwrap::indent` automatically prefixes empty lines with the
trimmed prefix (e.g. `//`, `///`, `*`), the post-processing `.replace`
calls across all code generators (`cpp`, `csharp`, `dart`, `golang`,
`java`, `kotlin`, `rust`, `swift`, `typescript`) are redundant and have
been removed.

@ma2bd ma2bd left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the PR. LGTM!

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