Skip to content

feat(run): add --command override to run arbitrary commands in a venv - #270

Merged
gnufede merged 2 commits into
masterfrom
gnufede/command-arg
Aug 27, 2026
Merged

feat(run): add --command override to run arbitrary commands in a venv#270
gnufede merged 2 commits into
masterfrom
gnufede/command-arg

Conversation

@gnufede

@gnufede gnufede commented Aug 26, 2026

Copy link
Copy Markdown
Member

Add a --command option to riot run that overrides the venv's baked command. When set, riot runs the override command in the venv (with the same prepare/env/pass-env semantics) instead of inst.command. {cmdargs} is substituted within the override if present.

This lets external runners (e.g. ddtest) run inside a riot venv and inherit riot's full env natively, without the caller reconstructing the activation env.

@gnufede
gnufede requested a review from a team as a code owner August 26, 2026 11:49
@gnufede
gnufede requested review from Kyle-Verhoog and removed request for a team August 26, 2026 11:49
@datadog-datadog-prod-us1-2

This comment has been minimized.

Add a --command option to `riot run` that overrides the venv's baked
command. When set, riot runs the override command in the venv (with the
same prepare/env/pass-env semantics) instead of inst.command. {cmdargs}
is substituted within the override if present.

This lets external runners (e.g. ddtest) run inside a riot venv
and inherit riot's full env natively, without the caller reconstructing
the activation env.
@gnufede
gnufede force-pushed the gnufede/command-arg branch from cb5c67d to fab9b69 Compare August 26, 2026 12:09
Round out the --command override feature: document usage in the
quickstart guide, add a release note, and cover the no-baked-command
run/skip paths with tests.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@gnufede
gnufede requested a review from brettlangdon August 27, 2026 10:25
@gnufede
gnufede merged commit ffafc06 into master Aug 27, 2026
29 checks passed
@gnufede
gnufede deleted the gnufede/command-arg branch August 27, 2026 13:55
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