hussein-awala commented on PR #34786: URL: https://github.com/apache/airflow/pull/34786#issuecomment-1749636013
> > @vincbeck @ashb Should we also update the [public interface document](https://airflow.apache.org/docs/apache-airflow/stable/public-airflow-interface.html) to include `has_access`? Maybe as a separate PR. > > I would definitely say yes. I would do it in a separate PR though, mainly because I dont really know how to explain besides saying: "Many plugins are using this decorator historically" :D One of you want to take a chance at it? I don't have a strong opinion on this, but as I understood in the original discussion, this decorator should not be in the public interface, but just maintain it because it was used since Airflow 1 for plugins, and we kept it in Airflow 2. Adding a deprecated method to the public interface doesn't make much sense for me. -- 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-unsubscr...@airflow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
