mik-laj commented on a change in pull request #12558:
URL: https://github.com/apache/airflow/pull/12558#discussion_r532050164
##########
File path: airflow/cli/commands/provider_command.py
##########
@@ -64,3 +63,25 @@ def provider_get(args):
def providers_list(args):
"""Lists all providers at the command line"""
print(_tabulate_providers(ProvidersManager().providers.values(),
args.output))
+
+
+def _tabulate_hooks(hook_items: Tuple[str, Tuple[str, str]], tablefmt: str):
Review comment:
There is something wrong with the type of this parameter. I guess this
is going to be a list of items because then we iterate over it.
> hook_item[1][3]
Then we read the index of the element which probably does not exist. Am I
right?
----------------------------------------------------------------
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]