vincbeck commented on code in PR #45678:
URL: https://github.com/apache/airflow/pull/45678#discussion_r1916709417


##########
newsfragments/aip-79.significant.rst:
##########
@@ -13,3 +13,13 @@ As part of this change the following breaking changes have 
occurred:
   - The property ``security_manager`` has been removed from the interface
 
   - The method ``filter_permitted_menu_items`` is now abstract and must be 
implemented
+
+* Types of change

Review Comment:
   Should we add a type for this use case? I guess there must be several cases 
where an interface that user might implement have breaking changes. Something 
like "Code interface change". Users who have a custom auth manager or custom 
executor and implement the auth manager/executor interface will face breaking 
change. Having this category might help them. I dont expect many users doing it 
but I am pretty sure there are some.



-- 
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-unsubscr...@airflow.apache.org

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

Reply via email to