fredthomsen commented on pull request #18542: URL: https://github.com/apache/airflow/pull/18542#issuecomment-966492874
I have rebased. I am sure someone somewhere out there wants that use case; however, the deprecation is going to be ugly no matter how or when you do it since this attribute is already in another object. Unless someone clamors loudly enough, then we can hold off until the next major release and don't have to deal with the deprecation for path. Basically, you'll have to twist my arm a bit hehe. ‐‐‐‐‐‐‐ Original Message ‐‐‐‐‐‐‐ On Monday, November 8th, 2021 at 3:53 PM, Daniel Standish ***@***.***> wrote: > thanks ***@***.***(https://github.com/fredthomsen) > yeah the reason i ask is because deprecations seem kindof ugly in helm and i figure, if we think we'll ultimately need path to be part of the host object we might as well just get that out of the way now. but i can't think of a reason why you'd want to specify different paths so i'm fine with leaving that as is. lemme know if you think otherwise ***@***.***(https://github.com/fredthomsen) or ***@***.***(https://github.com/jedcunningham) > > — > You are receiving this because you were mentioned. > Reply to this email directly, [view it on GitHub](https://github.com/apache/airflow/pull/18542#issuecomment-963563214), or [unsubscribe](https://github.com/notifications/unsubscribe-auth/AAGW3QETQNPXBC6DPEZDSETULA2ENANCNFSM5EZVY33Q). -- 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]
