vandonr-amz commented on PR #30495: URL: https://github.com/apache/airflow/pull/30495#issuecomment-1502145806
ah yes good examples. I switched to ast.parse() on Jarek's suggestion, it should handle this well. I'll add the multiline comment to test test file. About the if/else, I don't really know how we should handle it. Either we pre-imprt both branches, or we import none. I'd lean towards importing none because this is just an optim, and we want to do the 10% effort to handle 90% of the cases I'd say ? -- 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]
