Skip to content

[python] Fix inconsistent license headers#7841

Merged
JingsongLi merged 2 commits into
apache:masterfrom
XiaoHongbo-Hope:head_fix
May 13, 2026
Merged

[python] Fix inconsistent license headers#7841
JingsongLi merged 2 commits into
apache:masterfrom
XiaoHongbo-Hope:head_fix

Conversation

@XiaoHongbo-Hope
Copy link
Copy Markdown
Contributor

@XiaoHongbo-Hope XiaoHongbo-Hope commented May 13, 2026

Current license headers have several issues:

  • Corrupted text: you may not this file (missing "use")
  • Duplicated first lines in 2 files
  • Incorrect border lengths (79 or 81 # instead of 80)
  • 4 different comment styles coexisting: #### bordered, # , # , and """ docstring
  • Inconsistent URL indentation (4/5/6 spaces) and line wrapping across files

This PR unifies all headers to the standard borderless style used by pyiceberg, Airflow, Arrow, and other Apache Python projects. Also adds check_license_header.py to lint-python.sh CI to prevent future
inconsistencies.

Purpose

Tests

All .py files under pypaimon/ now use the same borderless license header
style consistent with pyiceberg, Airflow, Arrow, and other Apache Python
projects. Also fixes corrupted text, duplicated lines, and incorrect
border lengths. Adds check_license_header.py to lint-python.sh CI to
prevent future header inconsistencies.
@XiaoHongbo-Hope XiaoHongbo-Hope changed the title [python] Unify license headers to standard Apache style without borders [python] Unify license headers to standard Apache style May 13, 2026
@XiaoHongbo-Hope XiaoHongbo-Hope changed the title [python] Unify license headers to standard Apache style [python] Fix inconsistent license headers May 13, 2026
@JingsongLi
Copy link
Copy Markdown
Contributor

+1

@JingsongLi JingsongLi merged commit 2d269b8 into apache:master May 13, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants