Skip to content

infer output format from -o flag file extension - #7331

Merged
nwt merged 1 commit into
mainfrom
output-format
Sep 18, 2026
Merged

nwt merged 1 commit into
mainfrom
output-format

Conversation

@nwt

@nwt nwt commented Sep 18, 2026

Copy link
Copy Markdown
Member

If the -o flag specifies an output file name with a recognized extension and the -f flag is not specified, set the output format according to the extension.

Recognized extensions:

  • arrows
  • bsup
  • csup
  • csv
  • json
  • jsonl
  • ndjson
  • parquet
  • sup
  • text (line format)
  • tsv
  • txt (line format)

The only code changes here are in cli/outputflags/flags.go and sio/sio.go. All other changes are to documentation or tests.

Closes #7308.

@nwt
nwt requested a review from a team September 18, 2026 19:24
If the -o flag specifies an output file name with a recognized extension
and the -f flag is not specified, set the output format according to the
extension.

Recognized extensions:

  * arrows
  * bsup
  * csup
  * csv
  * json
  * jsonl
  * ndjson
  * parquet
  * sup
  * text (line format)
  * tsv
  * txt (line format)

The only code changes here are in cli/outputflags/flags.go and
sio/sio.go.  All other changes are to documentation or tests.
@nwt

nwt commented Sep 18, 2026

Copy link
Copy Markdown
Member Author

@philrz: Before I merge this I'd appreciate your giving it a look.

@philrz philrz left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me too. Thanks!

@nwt
nwt merged commit f6704a8 into main Sep 18, 2026
4 checks passed
@nwt
nwt deleted the output-format branch September 18, 2026 20:07
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.

Deriving output format from file extension in "-o" target

3 participants