New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

feat: expose ProcessPromise `fullCmd` and unique `id` #1035

Merged

antongolub merged 2 commits into google:main from antongolub:expose-full-cmd

Dec 25, 2024

Collaborator

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


          feat: expose ProcessPromise fullCmd and unique id

03f3fc1

relates google#1028

antongolub force-pushed the expose-full-cmd branch from 56854e7 to 03f3fc1 Compare

December 25, 2024 18:34

antongolub requested a review from antonmedv

December 25, 2024 18:35


          chore: size limit

66c817a

antongolub merged commit 90a4f24 into google:main

19 checks passed

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

Labels

None yet