Skip to content

Add prek hook to check NOTICE files for content #60699

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
dheerajturaga merged 1 commit into from
Jan 17, 2026
Merged

Add prek hook to check NOTICE files for content #60699

dheerajturaga merged 1 commit into from
Jan 17, 2026

Conversation

@potiuk
Copy link
Member

@potiuk potiuk commented Jan 17, 2026 •
edited
Loading

Related: #60540

Was generative AI tooling used to co-author this PR?
  • Yes (please specify the tool below)

Generated-by: Copilot + Claude Sonet 4.5 following the guidelines

  • Read the 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) is needed.
  • When adding dependency, check compliance with the ASF 3rd Party License Policy.
  • For significant user-facing changes create newsfragment: {pr_number}.significant.rst or {issue_number}.significant.rst, in airflow-core/newsfragments.

@boring-cyborg boring-cyborg bot added area:dev-tools backport-to-v3-1-test Mark PR with this label to backport to v3-1-test branch labels Jan 17, 2026
jason810496
jason810496 approved these changes Jan 17, 2026
View reviewed changes
Copy link
Member

@jason810496 jason810496 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice! LGTM.

@potiuk potiuk mentioned this pull request Jan 17, 2026
Open
1 task
@potiuk @jason810496
Add prek hook to check NOTICE files for content
28675e1 
Related: apache#60540

Co-authored-by: Jason(Zhe-You) Liu <68415893+jason810496@users.noreply.github.com>
@potiuk potiuk force-pushed the add-notice-file-prek-hook branch from d22ada3 to 28675e1 Compare January 17, 2026 13:34
@dheerajturaga dheerajturaga merged commit 49c3d0c into apache:main Jan 17, 2026
69 checks passed
github-actions bot pushed a commit that referenced this pull request Jan 17, 2026
@potiuk @jason810496
[v3-1-test] Add prek hook to check NOTICE files for content (#60699)
59466eb 
Related: #60540
(cherry picked from commit 49c3d0c)

Co-authored-by: Jarek Potiuk <jarek@potiuk.com>
Co-authored-by: Jason(Zhe-You) Liu <68415893+jason810496@users.noreply.github.com>
@github-actions
Copy link

github-actions bot commented Jan 17, 2026

Backport successfully created: v3-1-test

Status Branch Result
✅ v3-1-test PR Link

jason810496 added a commit to jason810496/airflow that referenced this pull request Jan 22, 2026
@potiuk @jason810496
Add prek hook to check NOTICE files for content (apache#60699)
d12acc5 
Related: apache#60540

Co-authored-by: Jason(Zhe-You) Liu <68415893+jason810496@users.noreply.github.com>
shreyas-dev pushed a commit to shreyas-dev/airflow that referenced this pull request Jan 29, 2026
@potiuk @jason810496 @shreyas-dev
Add prek hook to check NOTICE files for content (apache#60699)
9f14b97 
Related: apache#60540

Co-authored-by: Jason(Zhe-You) Liu <68415893+jason810496@users.noreply.github.com>
potiuk added a commit that referenced this pull request Jan 29, 2026
@potiuk @jason810496
[v3-1-test] Add prek hook to check NOTICE files for content (#60699)
cd56c1d 
Related: #60540
(cherry picked from commit 49c3d0c)

Co-authored-by: Jarek Potiuk <jarek@potiuk.com>
Co-authored-by: Jason(Zhe-You) Liu <68415893+jason810496@users.noreply.github.com>
potiuk added a commit that referenced this pull request Jan 29, 2026
@github-actions @potiuk @jason810496
[v3-1-test] Add prek hook to check NOTICE files for content (#60699) (#…
4ac42ec 
…60710)

Related: #60540
(cherry picked from commit 49c3d0c)

Co-authored-by: Jarek Potiuk <jarek@potiuk.com>
Co-authored-by: Jason(Zhe-You) Liu <68415893+jason810496@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@gopidesupavan gopidesupavan gopidesupavan approved these changes

@jason810496 jason810496 jason810496 approved these changes

@ashb ashb Awaiting requested review from ashb ashb is a code owner

@amoghrajesh amoghrajesh Awaiting requested review from amoghrajesh amoghrajesh is a code owner

@jscheffl jscheffl Awaiting requested review from jscheffl jscheffl is a code owner

@bugraoz93 bugraoz93 Awaiting requested review from bugraoz93 bugraoz93 is a code owner

@kaxil kaxil Awaiting requested review from kaxil kaxil is a code owner

Assignees

No one assigned

Labels

area:dev-tools backport-to-v3-1-test Mark PR with this label to backport to v3-1-test branch

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@potiuk @gopidesupavan @jason810496 @dheerajturaga