Skip to content

Let the file target of $> redirection be quoted#2038

Merged
rolandwalker merged 1 commit into
mainfrom
RW/spaces-in-shell-csv-redirect-target
Jul 24, 2026
Merged

Let the file target of $> redirection be quoted#2038
rolandwalker merged 1 commit into
mainfrom
RW/spaces-in-shell-csv-redirect-target

Conversation

@rolandwalker

@rolandwalker rolandwalker commented Jul 23, 2026

Copy link
Copy Markdown
Contributor

Description

Like .> to save parquet, $> targets should be allowed to be quoted, and to contain spaces if so.

Incidentally clarify in TIPS that $> defaults to CSV format.

Example:

SELECT * FROM orders $> "output.csv";

Checklist

  • I added this contribution to the changelog.md file.
  • I added my name to the AUTHORS file (or it's already there).
  • To lint and format the code, I ran
    uv run ruff check && uv run ruff format && uv run mypy --install-types .

@rolandwalker rolandwalker self-assigned this Jul 23, 2026
@rolandwalker
rolandwalker force-pushed the RW/spaces-in-shell-csv-redirect-target branch from aad457b to f8910db Compare July 23, 2026 13:29
Like .> to save parquet, $> targets should be allowed to be quoted, and
to contain spaces if so.

Incidentally clarify in TIPS that $> defaults to CSV format.
@rolandwalker
rolandwalker force-pushed the RW/spaces-in-shell-csv-redirect-target branch from f8910db to 259d7e1 Compare July 24, 2026 09:32
@rolandwalker
rolandwalker merged commit 8e83b22 into main Jul 24, 2026
11 checks passed
@rolandwalker
rolandwalker deleted the RW/spaces-in-shell-csv-redirect-target branch July 24, 2026 09:39
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