turbaszek commented on a change in pull request #12698:
URL: https://github.com/apache/airflow/pull/12698#discussion_r532188096
##########
File path: airflow/cli/commands/provider_command.py
##########
@@ -63,4 +69,15 @@ def provider_get(args):
def providers_list(args):
"""Lists all providers at the command line"""
- print(_tabulate_providers(ProvidersManager().providers.values(),
args.output))
+ console = Console()
Review comment:
> Since one line can contain more than one element
I agree and that has not changed. The only change here is removing rst
syntax from description and adding `|` between columns which helps when using
`cut -d "|"`
##########
File path: airflow/cli/commands/provider_command.py
##########
@@ -63,4 +69,15 @@ def provider_get(args):
def providers_list(args):
"""Lists all providers at the command line"""
- print(_tabulate_providers(ProvidersManager().providers.values(),
args.output))
+ console = Console()
+ table = SimpleTable(title="Installed providers")
+ table.add_column("Name")
+ table.add_column("Description")
+ table.add_column("Version")
+ for _, provider in ProvidersManager().providers.values():
+ table.add_row(
+ provider['package-name'],
+ _remove_rst_syntax(provider['description']),
Review comment:
The description has a url which can be helpful. However, I'm leaning
towards showing only provider + version. On the other hand that information is
also available via `airflow info`
----------------------------------------------------------------
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]
