potiuk commented on PR #53149: URL: https://github.com/apache/airflow/pull/53149#issuecomment-3065654755
> That one doesn't really worry me, as it's during dev time only, and ignored by IDEs and type checkers (that's what the typestubs are for, and if you look at them they are also very simple, it only needs one line per module) -- i.e. the metapath finder is only for Python when running from a checkout/editable install. I am simply very cautious about any kind of "Python magic" - not only becuase some tools that might not work with it, but also because it adds a lot of mental overload on contributors who would not be aware of it. There will likely be many questions from people who will not understand what's going on. I had a fair share of those things with Breeze - when you implement something that "just works" but details of it are hidden and not someothing you are familiar with on a daily basis, then what you end up with is that people will be asking many questions and expect their problems solved rather than trying to understand what's going on - because this will be unfamiliar and they will not want to spend time on understanding what's going on. This is what is so appealing about symlinks, because they are pretty much "standard" when it comes to coding - and you get exactly what you see - no hidden magic on how this or that import works in these or those circumstances. -- 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
