Taragolis commented on PR #36323: URL: https://github.com/apache/airflow/pull/36323#issuecomment-1864095959
I think it better to remove for consistency between providers - Not in all providers we have this importance note, only in this 4 providers, all other listed [here](https://airflow.apache.org/docs/apache-airflow-providers/core-extensions/notifications.html) do not have. - For new Notifiers which would add to providers this note doesn't make any sense It is good point to improve our Core documentation for Notifiers, e.g. there is only this [page](https://airflow.apache.org/docs/apache-airflow/stable/howto/notifications.html) in How-to guides, and there is not info about version added. So I guess it is better to move this page into the Core Concepts and add "version added" as follow up. -- 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]
