NewSmoke38 opened a new pull request, #60672:
URL: https://github.com/apache/airflow/pull/60672

   Description
   ---
   Added:
   `verify_notice_files.py`: script to validate NOTICE files in both source 
tree and distribution packages (.tar.gz, .whl)
   
   Verification checks:
   - Ensures required "Apache Software Foundation" text is present
   - Validates copyright year range ends with current year (2026)
   - Handles special cases (e.g., FAB provider with vendored Flask App Builder)
   
   **Integration:** Added verification steps to all 5 `README_RELEASE_*.md` 
files
   **Flags:** Supports --sources-only and --dist-only for flexible verification 
during development and release
   
   **Testing:**
   - Verified all 104 source NOTICE files successfully
   - Tested extraction and validation from real PyPI .whl packages
   - Validated error detection with incorrect copyright years
   - Confirmed proper exit codes (0 for success, 1 for failure)
   
   Future Work
   As mentioned in #60540
   
   - Exploring symbolic links to reduce duplication of NOTICE files across 200+ 
locations
   - Implementing pre-commit hooks to keep NOTICE files in sync automatically
   
   If this approach is approved, I'd be happy to work on these enhancements in 
a follow-up PR.
   
   closes: #60540
   
   
   ##### Was generative AI tooling used to co-author this PR?
   
   <!--
   If generative AI tooling has been used in the process of authoring this PR, 
please
   change below checkbox to `[X]` followed by the name of the tool, uncomment 
the "Generated-by".
   -->
   
   - [X] Yes 
   Generated-by: Claude Sonnet 4.5 on Antigravity IDE, following [Pull Request 
Guidelines](https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#pull-request-guidelines)
   
   ---
   
   * Read the **[Pull Request 
Guidelines](https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#pull-request-guidelines)**
 for more information. Note: commit author/co-author name and email in commits 
become permanently public when merged.
   * For fundamental code changes, an Airflow Improvement Proposal 
([AIP](https://cwiki.apache.org/confluence/display/AIRFLOW/Airflow+Improvement+Proposals))
 is needed.
   * When adding dependency, check compliance with the [ASF 3rd Party License 
Policy](https://www.apache.org/legal/resolved.html#category-x).
   * For significant user-facing changes create newsfragment: 
`{pr_number}.significant.rst` or `{issue_number}.significant.rst`, in 
[airflow-core/newsfragments](https://github.com/apache/airflow/tree/main/airflow-core/newsfragments).
   


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