Ma77Ball opened a new issue, #5056:
URL: https://github.com/apache/texera/issues/5056

   ### Feature Summary
   
   When the license-binary check fails in CI, the log lists only package names 
and versions. Maintainers have to look up two things by hand for each offender: 
the package's declared license, and which of the 9 per-module LICENSE-binary 
files to edit. Both are already available in the script's input data; they just 
aren't printed.
   
   ### Proposed Solution or Design
   
   Enrich the failure report in `bin/licensing/check_binary_deps.py` so each 
bullet shows the license and the file to edit:
    ```
       + [email protected]  (license: MIT) → add to frontend/LICENSE-binary
       - [email protected] → remove from frontend/LICENSE-binary
       ~ foo: LICENSE-binary=1.0  bundled=1.1 → update in 
frontend/LICENSE-binary
   ```
   
   License strings come from `3rdpartylicenses.json` (npm) and 
`pip-licenses.csv` (python). The target file is determined by 
`--license-binary` when set; otherwise, a per-ecosystem default. Covers 
transitive deps too, since the input files list everything bundled. No CI 
workflow changes.
   
   
   ### Affected Area
   
   Other


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to