rusackas opened a new pull request, #40395:
URL: https://github.com/apache/superset/pull/40395

   ### SUMMARY
   
   Translation `.po` files in Apache Superset require the ASF license header — 
the same header present in all other `.po` files in the repo (German, Chinese, 
etc.). Files initialized from the `.pot` template (e.g. via `msginit`) don't 
inherit this header, causing **License Check** CI failures on any PR that adds 
a new language.
   
   `backfill_po.py` now automatically prepends the header to any `.po` file 
that's missing it before beginning translation. This makes the tool 
self-healing: running it on a freshly-initialized `.po` file will fix the 
license header as a side effect, with no separate manual step required.
   
   **Related:** Discovered while fixing #40390 (Finnish) and #40391 (Thai) 
translation PRs that failed License Check.
   
   ### TESTING INSTRUCTIONS
   
   ```bash
   # Create a bare .po file without the header and confirm it gets fixed:
   python scripts/translations/backfill_po.py --lang <code> --dry-run
   # The script should print "Added ASF license header to ..." before 
proceeding.
   ```
   
   ### ADDITIONAL INFORMATION
   
   - [ ] Has associated issue:
   - [ ] Required feature flags:
   - [ ] Changes UI
   - [ ] Includes DB Migration
   - [ ] Introduces new feature or API
   - [ ] Removes existing feature or API
   
   🤖 Generated with [Claude Code](https://claude.com/claude-code)


-- 
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]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to