josh-fell commented on code in PR #30227: URL: https://github.com/apache/airflow/pull/30227#discussion_r1149972846
########## docs/apache-airflow-providers-dbt-cloud/operators.rst: ########## @@ -108,6 +115,7 @@ Use the :class:`~airflow.providers.dbt.cloud.sensors.dbt.DbtCloudJobRunAsyncSens status of a dbt Cloud job run asynchronously. This sensor will free up the worker slots since polling for job status happens on the Airflow triggerer, leading to efficient utilization of resources within Airflow. +:class:`~airflow.providers.dbt.cloud.sensors.dbt.DbtCloudJobRunAsyncSensor` is deprecated and will be removed in a future release. Please use :class:`~airflow.providers.dbt.cloud.sensors.dbt.DbtCloudJobRunSensor` and use the deferrable mode in that operator. Review Comment: Would you mind including this note in a `.. note::` directive? The formatting of the docs with a Note would put some emphasis around the deprecation. Let's remove the code snippet example below too since that example shows a deprecated sensor. -- 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]
