[GitHub] ashb commented on a change in pull request #4129: [AIRFLOW-3294] Update connections form and integration docs

[GitBox]

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_r231161317
 
 

 ##########
 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:
   Having looked at how this is rendered how about a middle ground - we list 
the classes in this doc, but just as links, rather than including the code 
docstrings?
   
   For example, in this screen shot the list and short description would stay, 
but the EmrAddStepsOperator wouldn't be in this doc?
   
   <img width="726" alt="screen shot 2018-11-06 at 15 11 25" 
src="https://user-images.githubusercontent.com/34150/48073272-50d22380-e1d6-11e8-801a-5a2e5401dbc3.png";>
   

----------------------------------------------------------------
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:
us...@infra.apache.org


With regards,
Apache Git Services