sjyangkevin commented on PR #54308: URL: https://github.com/apache/airflow/pull/54308#issuecomment-3174403921
> Email is not ideal, as it is not required in the base auth manager. While we can create a way to map user_name to user_id, we still need user_id as the primary identifier under the hook. Thanks for the feedback. You’re right. I had another look into that yesterday and I also found that may depend on the auth manager. I think map user name to user id under the hook is a good idea. > As for the CI failure, there are a few hooks to run, so I would suggest just running `pre-commit run --all-files` and wait for the first time, and list down the ones that fail in case we need to run it again. I did run `pre-commit run —all-files` but probably I can try again by first cleaning cache 🤔 -- 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]
