dheerajturaga commented on PR #62549:
URL: https://github.com/apache/airflow/pull/62549#issuecomment-3988165458

   > It looks good! We did similar thing for password here. 
https://github.com/dheerajturaga/airflow/blob 
/7721fe7b3ec609c21ac792670c89be3959caf227/airflow-ctl/src
   > 
   > /airflowctl/ctl/cli_config.py#L198 Do you think we should move to command, 
in that case we should clean it. They seems doing the same thing. I am okay 
with both approach since as is custom to the auth command
   > 
   > We can merge afterwards. It is just to ensure we are only doing once :)
   
   Yea I think its better in the command. I made some updates. Let me know if 
you like them :) @bugraoz93 


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to