feluelle commented on PR #28721: URL: https://github.com/apache/airflow/pull/28721#issuecomment-1411582283
Yes, good catch @ashb and @josh-fell thank you for the heads-up. That is a tricky one. I would say we merge the other PR first and I will just update my PR and docs to reflect the changes i.e. that from now on you could use certain fields such as login, password, etc. The current documentation also states that the fields are required, which they are not even with my change using the URL. But I would like to bring @atharvmishra1999 to the discussion. How do you think we should handle the collision? -- 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.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
