Skip to content

Conversation

@andy1li
Copy link
Member

@andy1li andy1li commented Nov 13, 2025

Before:

The single quotes are not testing "backslash within double quotes" at all.

image

After:

image

Note

Use %q to quote file paths for the custom cat command in testQ5 instead of manual single quotes.

  • Tests (internal/stage_q5.go):
    • Update inputs construction in testQ5 to use Go %q for quoting file paths when invoking CUSTOM_CAT_COMMAND, replacing manual single-quoting.
    • Keeps expected outputs unchanged; only command construction is adjusted.

Written by Cursor Bugbot for commit a444765. This will update automatically on new commits. Configure here.

@andy1li andy1li self-assigned this Nov 13, 2025
@andy1li andy1li added the regenerate-fixtures Trigger a CI job to regenerate fixtures label Nov 13, 2025
@github-actions
Copy link

Triggered a Github Actions job to update fixtures.

@github-actions github-actions bot removed the regenerate-fixtures Trigger a CI job to regenerate fixtures label Nov 13, 2025
@andy1li andy1li changed the title Refactor testQ5 function to remove unnecessary single quotes Remove single quotes in #gu3 Backslash within double quotes Nov 13, 2025
@andy1li andy1li added the regenerate-fixtures Trigger a CI job to regenerate fixtures label Nov 13, 2025
@github-actions
Copy link

Triggered a Github Actions job to update fixtures.

@github-actions github-actions bot removed the regenerate-fixtures Trigger a CI job to regenerate fixtures label Nov 13, 2025
@andy1li andy1li merged commit 54bc945 into main Nov 13, 2025
5 checks passed
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.

4 participants