[GitHub] [airflow] michaelosthege opened a new pull request, #25197: Include missing mention of `external_executor_id` in docstring

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

michaelosthege opened a new pull request, #25197:
URL: https://github.com/apache/airflow/pull/25197

   As suggested by @potiuk in https://github.com/apache/airflow/issues/24526#issuecomment-1190658878 this fixes missing information in the docstring for the `sql_engine_collation_for_ids` setting.


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

[GitHub] [airflow] potiuk merged pull request #25197: Include missing mention of `external_executor_id` in docstring

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

potiuk merged PR #25197:
URL: https://github.com/apache/airflow/pull/25197


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

[GitHub] [airflow] boring-cyborg[bot] commented on pull request #25197: Include missing mention of `external_executor_id` in docstring

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

boring-cyborg[bot] commented on PR #25197:
URL: https://github.com/apache/airflow/pull/25197#issuecomment-1191903543

   Awesome work, congrats on your first merged pull request!
   


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