Add command accessors to CommandGroupBuilder
#27
Closed
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Hello! I'm implementing
command-error
, a crate that provides nicer error messages for commands, like this:I'd like to add optional support for the
command-group
crate (9999years/command-error#2). Because traits can only be implemented once per type (and I think a generic trait would have poor ergonomics here), I'm attempting toimpl command_error::CommandExt for command_group::builder::CommandGroupBuilder<'_, std::process::Command>
, but it's not possible to access the underlyingCommand
(to produce the name of the program that was run for error messages) with the current API.Therefore, this PR adds getters to access the
Command
field ofCommandGroupBuilder
.