peczenyj / GDPR-IAB-TCFv2
95%
main: 96%

LAST BUILD BRANCH: devel
DEFAULT BRANCH: main
Repo Added
Files 18
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

LAST BUILD ON BRANCH chore/remove-json-array-flag
branch: chore/remove-json-array-flag
CHANGE BRANCH
x

coverage: 95.223% (+0.8%) from 94.404%
25491282391

Pull #62

github

peczenyj 
chore(cli): remove --json-array; default JSONL output is enough

Removes the --json-array flag from both `iabtcfv2 dump` and
`iabtcfv2 validate`.  The default JSON Lines output (one JSON
object per line) is the universal format -- every text-processing
tool reads it directly, and `jq -s .` wraps it into a single
JSON array when callers actually need that shape.

Carrying --json-array meant carrying state-tracking code in both
subcommands: opening/closing brackets gated on quiet/text/fail-fast
combinations, comma-injection between records, the dead `_indent`
helper, plus a mutually-exclusive check between --text and
--json-array in validate.  All of that goes; the per-record print
now reduces to `print $j->encode($data) . "\n"`.

Migration: replace `iabtcfv2 dump --json-array ...` with
`iabtcfv2 dump ... | jq -s .` (or any equivalent JSONL-to-array
tool).  POD examples updated to show the jq pipeline.

Test plan:
  - t/10-cli-iabtcfv2.t: dropped the --json-array functional test;
    replaced with a JSONL-shape assertion (multiple TC strings ->
    multiple newline-separated JSON objects).
  - help-text assertion for --json-array dropped; replaced with
    --pretty.
  - Full suite passes (16536 tests; xt/critic and xt/tidy green).

This is technically a breaking CLI change three commits after
v0.370 shipped, but the migration path is one shell pipe.  Worth
the simplification.
Pull Request #62: chore(cli): remove --json-array; JSONL is enough

3 of 3 new or added lines in 1 file covered. (100.0%)

897 of 942 relevant lines covered (95.22%)

12386.56 hits per line

Relevant lines Covered
942 RELEVANT LINES 897 COVERED LINES
12386.56 HITS PER LINE
Source Files on chore/remove-json-array-flag

Recent builds

Builds Branch Commit Type Ran Committer Via Coverage
25491282391 chore/remove-json-array-flag chore(cli): remove --json-array; default JSONL output is enough Removes the --json-array flag from both `iabtcfv2 dump` and `iabtcfv2 validate`. The default JSON Lines output (one JSON object per line) is the universal format -- every text-proce... Pull #62 peczenyj github
95.22
See All Builds (549)