Skip to content

feat: expose ProcessPromise fullCmd and unique id#1035

Merged
antongolub merged 2 commits intogoogle:mainfrom
antongolub:expose-full-cmd
Dec 25, 2024
Merged

feat: expose ProcessPromise fullCmd and unique id#1035
antongolub merged 2 commits intogoogle:mainfrom
antongolub:expose-full-cmd

Conversation

@antongolub
Copy link
Copy Markdown
Collaborator

@antongolub antongolub commented Dec 25, 2024

relates #1028

const foo = '#bar'
const baz = 1
const p = $`echo ${foo} --t ${baz}`
p.cmd       // "echo $'#bar' --t 1")
p.fullCmd   // "set -euo pipefail;echo $'#bar' --t 1"
  • Tests pass
  • Appropriate changes to README are included in PR

@antongolub antongolub requested a review from antonmedv December 25, 2024 18:35
@antongolub antongolub merged commit 90a4f24 into google:main Dec 25, 2024
@antongolub antongolub deleted the expose-full-cmd branch December 25, 2024 19:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@antonmedv antonmedv Awaiting requested review from antonmedv

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@antongolub