potiuk commented on issue #36117:
URL: https://github.com/apache/airflow/issues/36117#issuecomment-1848947322

   Tested mssql - all looks good.
   
   One more thing I noticed - the removed `daskexecutor` has not been prepared. 
 We have this in the release process, so it's easy to miss:
   
   ```
   NOTE! When you want to release a provider marked for removal (needed in 
order to prepare last release of the provider), package for the provider will 
not be prepared when you prepare documentation for all providers - you have to 
specifically use the provider name in a separate command. For example to 
prepare documentation for removed.provider provider marked for removal you need 
to run separately this command:
   
   breeze release-management prepare-provider-packages --package-format both 
removed.provider
   ```
   
   But maybye we will change it in the future - we have more removed providers 
coming. So possibly we shoudl automatically add removed providers to the list 
of providers prepared (both for documentation and package building - so that we 
do not have to remember to prepare them. That was probably bad idea to not add 
them in the first place.
   
   @eladkal -> I guess we should include rc2 (even if there was no rc1) for 
`daskexecutor`.
   
   


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