Skip to content

all: os/exec => internal/xec#988

Merged
abhinav merged 2 commits intomainfrom
xec-everywhere
Jan 11, 2026
Merged

all: os/exec => internal/xec#988
abhinav merged 2 commits intomainfrom
xec-everywhere

Conversation

@abhinav
Copy link
Owner

@abhinav abhinav commented Jan 11, 2026

Switch all uses of "os/exec" to "internal/xec"
to centralize all external command execution.

Besides making testing easier,
this allows for shipping functionality like
"all subcommands will get this environment variable set"
(which is something I'm planning on adding).

  • Vet all call sites again to make sure that nothing that was previously surfacing stderr is now consuming it.

[skip changelog]: no user facing changes

Switch all uses of "os/exec" to "internal/xec"
to centralize all external command execution.
@abhinav
Copy link
Owner Author

abhinav commented Jan 11, 2026

This change is part of the following stack:

Change managed by git-spice.

@abhinav abhinav marked this pull request as ready for review January 11, 2026 18:54
@abhinav abhinav merged commit 763b5c5 into main Jan 11, 2026
22 checks passed
@abhinav abhinav deleted the xec-everywhere branch January 11, 2026 18:54
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.

1 participant