Skip to content

Add comment for cadwyn PR allowing us to remove the limit#61941

Merged
potiuk merged 1 commit intoapache:mainfrom
potiuk:add-link-to-pr-removing-limit
Feb 15, 2026
Merged

Add comment for cadwyn PR allowing us to remove the limit#61941
potiuk merged 1 commit intoapache:mainfrom
potiuk:add-link-to-pr-removing-limit

Conversation

@potiuk
Copy link
Member

@potiuk potiuk commented Feb 15, 2026


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

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

@potiuk
Copy link
Member Author

potiuk commented Feb 15, 2026

Good job @shahar1 on zmievsa/cadwyn#327 -> just for posterity, and in case it takes a bit of time, I think it's a good practice to always have comment linking to issues / prs that cause upper-binding (so that it is very easy to check without going back in commit history/PRs to find out if we can remove things :) ).

@shahar1
Copy link
Contributor

shahar1 commented Feb 15, 2026

Good job @shahar1 on zmievsa/cadwyn#327 -> just for posterity, and in case it takes a bit of time, I think it's a good practice to always have comment linking to issues / prs that cause upper-binding (so that it is very easy to check without going back in commit history/PRs to find out if we can remove things :) ).

Noted, thanks for the feedback!

@potiuk potiuk merged commit 3cec81b into apache:main Feb 15, 2026
101 checks passed
@potiuk potiuk deleted the add-link-to-pr-removing-limit branch February 15, 2026 12:08
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.

2 participants