eladkal commented on pull request #15510:
URL: https://github.com/apache/airflow/pull/15510#issuecomment-838054629


   > And it think we _could_ optionally first fix the `if` statement by adding 
an `else return val` in the current provider major >version, before removing 
`clean_bool` in next
   
   I agree. If we can preserve backward compatibility with deprecation notice 
we should. we can remove `clean_bool` in the in a followup major release 
(whenever it may be)
   
   In any case the steps to bump the major version of provider are:
   
   1. bump [provider 
yaml](https://github.com/apache/airflow/blob/master/airflow/providers/odbc/provider.yaml#L24)
   2. add detailed explanation about the change in the [provider 
changelog](https://github.com/apache/airflow/blob/master/airflow/providers/odbc/CHANGELOG.rst)
   @potiuk did I miss something?
   


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

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to