Skip to content

h3i: make the command-line tool optional - #2836

Open
LPardue wants to merge 1 commit into
cloudflare:masterfrom
LPardue:h3i-cli-feat
Open

LPardue wants to merge 1 commit into
cloudflare:masterfrom
LPardue:h3i-cli-feat

Conversation

@LPardue

@LPardue LPardue commented Oct 15, 2026

Copy link
Copy Markdown
Contributor

Projects that use h3i only as a library had to build the packages that
its command-line tool needs. For example, the tokio-quiche tests use
h3i to send requests, but they also built the interactive prompt
library and its terminal support. That's 14 packages the tests never
used.

The command-line tool and its interactive prompts now sit behind a new
"cli" feature. The feature is on by default, so installing h3i still
gives you the tool. Inside this workspace, h3i is used without the
tool.

Code that uses the prompts module with default features turned off
must now turn on "cli".

The tool's async mode also needs the "rt" feature of tokio. It worked
only because tokio-quiche happened to turn that feature on. h3i now
asks for it itself.

Projects that use h3i only as a library had to build the packages that
its command-line tool needs. For example, the tokio-quiche tests use
h3i to send requests, but they also built the interactive prompt
library and its terminal support. That's 14 packages the tests never
used.

The command-line tool and its interactive prompts now sit behind a new
"cli" feature. The feature is on by default, so installing h3i still
gives you the tool. Inside this workspace, h3i is used without the
tool.

Code that uses the prompts module with default features turned off
must now turn on "cli".

The tool's async mode also needs the "rt" feature of tokio. It worked
only because tokio-quiche happened to turn that feature on. h3i now
asks for it itself.
@LPardue
LPardue requested a review from a team as a code owner October 15, 2026 20:00
@jrouviere

Copy link
Copy Markdown
Contributor

/bonk

@ask-bonk

ask-bonk Bot commented Oct 16, 2026

Copy link
Copy Markdown

LGTM!

github run

@jrouviere jrouviere 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.

LGTM
Nice to see you again!

This branch has not been deployed

No deployments
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