test: cover UTF-8 file pipe regressions#243
Conversation
|
@EfeDurmaz16 is attempting to deploy a commit to the Vercel Labs Team on Vercel. A member of the Team first needs to authorize it. |
|
Great find! just ran into this - thanks for opening this pr |
|
Looks great. Please sign your commit and run pnpm changeset! |
|
Thanks! I’ll sign the commit and add a changeset now. |
4170561 to
5814e10
Compare
|
Done. Added the changeset and updated the PR to use a verified signed commit. |
5814e10 to
ed6c6d7
Compare
|
Done. I amended the commit with I also re-ran the focused regression coverage for this PR:
The remaining red checks are Vercel preview deployment authorization checks. |
|
Please re-push with a signed commit |
Signed-off-by: EfeDurmaz16 <efebarandurmaz05@gmail.com>
ed6c6d7 to
cd4f1a5
Compare
|
Re-pushed with a GitHub-verified SSH-signed commit: cd4f1a5. No content changes beyond rewriting the commit signature. The remaining red checks are Vercel deployment authorization checks. |
Summary
Adds regression coverage for the UTF-8 file-read cases discussed in #242:
catdecodeBytesToUtf8(ctx.stdin)for text operationsVerification
pnpm --dir packages/just-bash exec vitest run src/commands/jq/jq.utf8-stdin.test.ts src/custom-commands.test.tspnpm --filter just-bash typecheckpnpm lintpnpm --filter just-bash knippnpm --filter just-bash buildRefs #242