search

jhspetersson / fselect

Find files with SQL-like queries
https://fselect.rocks
Apache License 2.0
4.02k stars 85 forks source link

json output missing comma separator #121

Closed jgranduel closed 2 years ago

jgranduel commented 2 years ago

versions: fselect-0.7.7.. latest (0.7.9)

The comma between object in the output array is not present.

$ touch a b
$  fselect name from .  into json
[{"Name":"a"}{"Name":"b"}]
                    ^
$  fselect name from .  into json | jq
parse error: Expected separator between values at line 1, column 14

With fselect-0.7.6, the JSON output was still correct AFAIK.

rickhg12hs commented 2 years ago

Ref: #120 Fixed after regression with 9fdcbadaf91a82a20629ab6d659de460a9eab73e.

jgranduel commented 2 years ago

Thanks, I hadn't seen #120 as you guessed.