turbaszek commented on a change in pull request #8440: Add airflow connection 
lookup command
URL: https://github.com/apache/airflow/pull/8440#discussion_r410740772
 
 

 ##########
 File path: airflow/cli/commands/connection_command.py
 ##########
 @@ -15,29 +15,41 @@
 # specific language governing permissions and limitations
 # under the License.
 """Connection sub-commands"""
-import reprlib
+from typing import List
 from urllib.parse import urlunparse
 
 from sqlalchemy.orm import exc
 from tabulate import tabulate
 
+from airflow.hooks.base_hook import BaseHook
 from airflow.models import Connection
 from airflow.utils import cli as cli_utils
 from airflow.utils.session import create_session
 
 
+def _tabule_connection(conns: List[Connection], tablefmt: str):
 
 Review comment:
   ```suggestion
   def _tabulate_connection(conns: List[Connection], tablefmt: str):
   ```

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to