On Sat, Jun 13, 2026 at 6:07 PM Sami Imseih <[email protected]> wrote:
> This comment is not needed. > > But I think we should add a comment explaining the condition here. > Done, replaced with a short comment above the condition and dropped the inner block comment. Should we just add tests for all the output types? > Added. v2 has column-reorder tests for text and csv (before the JSON section) plus subset and reorder tests for JSON in the JSON section. Thanks for the review, Sami. And thanks Andrew for the +1. v2 attached.
v2-0001-COPY-TO-FORMAT-JSON-respect-column-list-order.patch
Description: Binary data
