Skip to content

Print more information when repeating -V option - #507

Draft
kvid wants to merge 1 commit into
wireviz:release/v0.4.2-rcfrom
kvid:print_versions
Draft

kvid wants to merge 1 commit into
wireviz:release/v0.4.2-rcfrom
kvid:print_versions

Conversation

@kvid

@kvid kvid commented Jun 17, 2026

Copy link
Copy Markdown
Collaborator

It is challenging for users to collect information about their environment as described in the contributing guidelines.

#387 (comment) suggests expanding WireViz to collect this information. Inspired by this suggestion, this PR suggests a way that WireViz collects such information when the CLI option -VV is used.

@kvid kvid mentioned this pull request Jun 18, 2026

@tbornon-sts tbornon-sts left a comment •

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Tested on WSL and it works great.

~ wireviz -VV

WireViz 0.4.2-dev
Python 3.12.3 (main, Jun 19 2026, 12:46:00) [GCC 13.3.0]
Graphviz 2.43.0
Platform: Linux-5.15.167.4-microsoft-standard-WSL2-x86_64-with-glibc2.39
Distribution: ('Ubuntu 24.04.4 LTS', 'Ubuntu', '24.04.4 LTS (Noble Numbat)', 'ubuntu', '24.04', 'noble', 'debian')

Just not sure regarding contributing message, I've left a comment below

Comment thread docs/CONTRIBUTING.md
- **[meta]** For issues about the development or contribution process.
- Please include enough information in the description to enable another user to reproduce any error state described in your issue:
- The versions of your WireViz, Graphviz (`dot -V`), Python (`python -V`), and operating system.
- The versions of your WireViz (`wireviz -VV`), Graphviz (`dot -V`), Python (`python -V`), and operating system.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Is it relevant to ask for Graphviz version and python version since they are now included of the output of wireviz -VV ?

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