The GitHub Actions job "Required Checks" on texera.git/main has succeeded.
Run started by GitHub user aicam (triggered by aicam).

Head commit for run:
6060877a4d0a94ba9b3ce186334d1a230055c585 / Matthew B. <[email protected]>
ci: enrich check_binary_deps failure report with license + target file (#5057)

### What changes were proposed in this PR?
Enrich the failure output of `bin/licensing/check_binary_deps.py` with
two pieces of context that previously required manual lookup for every
offending package: the dep's declared license string (read from
`3rdpartylicenses.json` for npm/agent-npm or `pip-licenses.csv` for
python) and the per-module `LICENSE-binary` file to edit. Each bullet
now renders as `+ name@version (license: X) → add to <file>` for new
packages, `→ remove from <file>` for stale, and `→ update in <file>` for
drift, with the `ACTION REQUIRED` block naming the same file. Transitive
deps are covered the same way because the input files already list
everything bundled. No CI workflow changes; The script's exit semantics
and aggregation behavior are unchanged.
  ### Any related issues, documentation, or discussions?
Closes: #5056  
### How was this PR tested?
Added five unit tests in `bin/licensing/test_check_binary_deps.py`
covering license rendering, target-file rendering, the jar no-license
case, stale and drift hints, and the no-target-file fallback. Full suite
of 34 tests passes locally via `python3 -m unittest discover -s
bin/licensing -p "test_*.py"`. Also smoke-tested by running the script
against a synthetic `3rdpartylicenses.json` and confirming the enriched
bullets render correctly for new, stale, and drifted entries.
### Was this PR authored or co-authored using generative AI tooling?
  Co-authored with Claude Opus 4.7 in compliance with ASF

---------

Co-authored-by: ali risheh <[email protected]>

Report URL: https://github.com/apache/texera/actions/runs/25946645322

With regards,
GitHub Actions via GitBox

Reply via email to