josh-fell commented on PR #30176: URL: https://github.com/apache/airflow/pull/30176#issuecomment-1828970904
Reviving this after finally getting out of a large hole. @fritz-astronomer Yes, this approach is very much inspired by the Astro SDK. @potiuk I can confirm this works for multi-line strings. @sethwoodworth Initially I was thinking about keeping the existing default for `append_env`. However, I think I will ultimately do away with the `push_op_kwargs_to_env` parameter. It feels a bit quirky and can lead to less explicit scripting. For an "introduction" of this decorator, it seems a bit more pragmatic to not gum-up this PR but rather discuss that particular feature in a future PR. WDYT? -- 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]
