Skip to content

Collect changes for v0.4.2-rc - #500

Draft
kvid wants to merge 1 commit into
masterfrom
release/v0.4.2-rc
Draft

kvid wants to merge 1 commit into
masterfrom
release/v0.4.2-rc

Conversation

@kvid

@kvid kvid commented May 27, 2026 •

Copy link
Copy Markdown
Collaborator

Unfortunately, the v0.5 is still not close to finished, and I therefore in the meantime prepare a small release by collecting into this branch some PRs with bug-fixes and other small changes that should be easy to merge into the v0.5-dev branch later.

@martinrieder

martinrieder commented Oct 12, 2026 •

Copy link
Copy Markdown
Contributor
  • TODO: Any other candidates?

@SomethingNew71 proposed some contributions from his fork at ClassicMiniDIY/WireViz in #316 (comment)

So far I've merged in #321, #234, #367, #444, #379, #357, #492, #496, #495, #473 and #443, added tests, and fixed around 20 bugs (#497 and #510 included). That's all in v1.0.0. Still GPLv3, it's based on master (not dev), and each commit links back to the original PR or issue.

The fork used these existing upstream PRs as the foundation, so we might consider cross-checking and adding some of them. I added checkmarks for easier comparison with the list above:

Some of the following bugfixes come additional, but they do not have an upstream issue or PR to be listed here anyway:

Many more changes were applied in additional batches, each covering multiple issues at once. These would need to be broken down into smaller pieces for reviewability. See 2026-10-02-upstream-issue-triage.md and 2026-10-02-batch-c-design.md for details.

@ggrossetie

Copy link
Copy Markdown

Hi @kvid,

Regarding "Any other candidates?", I'd like to propose three small PRs, all based on master and mergeable without conflicts:

Some context: WireViz is one of the diagram libraries available on Kroki, which renders diagrams from textual descriptions over HTTP. Kroki pipes the YAML into WireViz and reads the SVG back from stdout. Since this isn't supported upstream, we've been maintaining a fork (yuzutech/WireViz) since 2023, still based on v0.3.2. Getting these changes into v0.4.2 would allow us to drop the fork and offer an up-to-date WireViz to Kroki users.

The changes are limited to the CLI and print calls: the library API and the generated files are unchanged. Since wv_cli.py is nearly identical on dev, they should also be easy to port to v0.5 later. I rewrote #321 from scratch on top of master for this purpose; the previous dev-based version was much larger. As mentioned above, an earlier version of #321 is also included in ClassicMiniDIY/WireViz, so it seems other users need it too.

I'm happy to address any feedback, or to rebase onto release/v0.4.2-rc if you prefer.

@martinrieder

martinrieder commented Oct 17, 2026 •

Copy link
Copy Markdown
Contributor

@ggrossetie No doubt that #513 qualifies for this bugfix release, but I think that changing the output stream in #321 and #512 are significant changes. The latter should rather be in 0.5, but anyway I vote for this when it improves error detection for nanangp/vscode-wireviz-preview#7, which would work even better with this:

Additionally we could add some documentation or template fixes without risk:

Also I suggest to cherry-pick 22c5250 that was suggested here:

@ggrossetie

Copy link
Copy Markdown

Thanks @martinrieder for taking the time to consider my changes for this release.

Fair point. I agree #321 is a larger change (it also touches the Python API, e.g. generate_html_output and embed_svg_images_file), so it makes sense to target 0.5.

#512 is much narrower: generated files are unchanged, only diagnostic messages move from stdout to stderr. The only consumers affected would be tools parsing stdout to capture warnings, and they get a cleaner contract in return (stdout for output, stderr for diagnostics), which is exactly what helps nanangp/vscode-wireviz-preview#7. Since we're still in 0.x, I think it's acceptable in 0.4.2 as long as it's called out explicitly in the CHANGELOG, something like:

"Breaking: warnings and progress messages are now written to stderr instead of stdout. If you were capturing them from stdout, read stderr instead. wireviz -V still prints to stdout."

@kvid
kvid force-pushed the release/v0.4.2-rc branch from a591490 to 7ef9c6b Compare October 25, 2026 18:13
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.

3 participants