potiuk commented on PR #55894: URL: https://github.com/apache/airflow/pull/55894#issuecomment-3410209869
> Wouldn't be enough to rename this argument (not sure it is that simple for backward compatibility), to make it clear it modifies sys.path or add additional argument append_bundle_to_syspath or similar? As mentioned above - that opens up for potential side-effects in places where the path should not be added - so I think (@uranusjr ?) that explicitly adding the path in the places where we know we are about to exit right after is likely a better approach. In this case path modification is really tied with the place where it is used, not the DagBag itself. -- 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]
