ashb commented on a change in pull request #4129: [AIRFLOW-3294] Update connections form and integration docs URL: https://github.com/apache/incubator-airflow/pull/4129#discussion_r230731775
########## File path: docs/integration.rst ########## @@ -1011,3 +1011,13 @@ QuboleFileSensor '''''''''''''''' .. autoclass:: airflow.contrib.sensors.qubole_sensor.QuboleFileSensor + +QuboleCheckOperator +''''''''''''''''''' + +.. autoclass:: airflow.contrib.operators.qubole_check_operator.QuboleCheckOperator + +QuboleValueCheckOperator +'''''''''''''''''''''''' + +.. autoclass:: airflow.contrib.operators.qubole_check_operator.QuboleValueCheckOperator Review comment: We need to work out what we put in this section more generally. In the first case _all_ operators/hooks/sensors should be in the `docs/code.rst` as that is the complete list. Secondly I'm not sure that listing operators in here is all that useful - I think this doc should be limited to interesting integrations, i.e. ones that need settings or configuring (like the reverse proxy) rather than just A second list of classes with no further description of them. ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services
