Re: [PR] fix for role and permission count in airflow export cli command [airflow]

2024-01-26 Thread via GitHub
potiuk merged PR #36589: URL: https://github.com/apache/airflow/pull/36589 -- 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. To unsubscribe, e-mail: commits-unsubscr...@airflow.a

Re: [PR] fix for role and permission count in airflow export cli command [airflow]

2024-01-23 Thread via GitHub
Bowrna commented on PR #36589: URL: https://github.com/apache/airflow/pull/36589#issuecomment-1907437156 this code can be be reviewed and checked if its ok to merge :) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use th

Re: [PR] fix for role and permission count in airflow export cli command [airflow]

2024-01-18 Thread via GitHub
Bowrna commented on code in PR #36589: URL: https://github.com/apache/airflow/pull/36589#discussion_r1458255173 ## airflow/providers/fab/auth_manager/cli_commands/role_command.py: ## @@ -176,7 +176,8 @@ def roles_export(args): kwargs = {} if not args.pretty else {"sort_keys

Re: [PR] fix for role and permission count in airflow export cli command [airflow]

2024-01-04 Thread via GitHub
vincbeck commented on code in PR #36589: URL: https://github.com/apache/airflow/pull/36589#discussion_r1441856501 ## airflow/providers/fab/auth_manager/cli_commands/role_command.py: ## @@ -176,7 +176,8 @@ def roles_export(args): kwargs = {} if not args.pretty else {"sort_ke

[PR] fix for role and permission count in airflow export cli command [airflow]

2024-01-04 Thread via GitHub
Bowrna opened a new pull request, #36589: URL: https://github.com/apache/airflow/pull/36589 related: #36580 This has fix only for the airflow roles export command message. --- **^ Add meaningful description above** Read the **[Pull Request Guideli