[GitHub] [airflow] potiuk commented on pull request #24884: Upgrade FAB to 4.1.3

[GitBox <gi...@apache.org>]

potiuk commented on PR #24884:
URL: https://github.com/apache/airflow/pull/24884#issuecomment-1176793696

   You can't update constraints in the same PR because constraints are in orphaned branch (constraints-main). 
   
   The Constraints main should be refreshed at merge time via https://github.com/apache/airflow/blob/main/dev/REFRESHING_CI_CACHE.md#manually-generating-constraint-files
   
   This is only really needed when you have the == change (or otherwise chagne that introduces range of dependency that does not overlap with the previous range). This is the only case that is not (yet) handled automatically (it could be, but we would have to create a PR automatically to the `constraints-main` branch and let them be merged at the same time.
   
   Effectively it means that the person (committer) who merges it, should prepare constraints just before merge and do the push at the same time as merge (either way it needs a bit synchronized action).  You cannot push to the branch as you are not a committer. 


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org