potiuk commented on a change in pull request #18883:
URL: https://github.com/apache/airflow/pull/18883#discussion_r726621873



##########
File path: CHANGELOG.txt
##########
@@ -125,6 +125,7 @@ Improvements
 - Refactor: ``SKIPPED`` should not be logged again as ``SUCCESS`` (#14822)
 - Remove  version limits for dnspython (#18046, #18162)
 - Accept custom run ID in TriggerDagRunOperator (#18788)
+- Align the default version with Facebook business SDK (#18883)

Review comment:
       > So where do we want to describe the breaking changes? I am afraid that 
the title alone is not enough in many cases.
   
   Sure - we need to figure it out. It's not done yet. We need:
   
   1) a way to do it
   2) a way to flag if not done for all PR
   3) a documentation for contributors to follow 
   4) a way to automate it to bring it to changelog automatically
   
   Until it's not done, it's just random advice, because no-one (including 
other committers) will follow it.
   
    I think about implementing towncrier for that and documenting it, but until 
it is done, I think either someone does it (all points) or we just follow the 
currnet process (where release manager does it), because we have no 
documentation for it (unless you tell me otherwise and documentation is there)? 




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