Closed avmaksimov closed 3 years ago
@Pectojin , please merge it if you don't mind and there are no errors in it,
Oh, this merge request was against the utf-8-sig
branch, which marked this issue as closed when I deleted the branch.
I'm looking into making a few changes, but overall this looks very good.
Some thoughts I've had while testing:
I wanna move the Loaded 1 parameters from file
message over to verbose logging so it doesn't break the flow of duc xxxxx | jq
I'm playing around with merging all the dicts inside the array so the data is more easily accessible. Right now I'm just unwrapping the top level array using jq
to no benefit jq '.[]["Files"].ID'
, which could just be jq '.["Files"].ID'
I'll just finish testing then I can merge your change in.
I guess merging it doesn't really work for repeating types such as the logs. I'll just leave it as is. Still a big improvement over having to parse plaintext.