potiuk commented on PR #53506: URL: https://github.com/apache/airflow/pull/53506#issuecomment-3096237597
> Looking promising, thanks! I'll dig in to this a bit more Indeed. It is surprisingly simple, and I was a bit (just a bit) surprised how little code and changes it needed - but I walked through all the use cases I can imagine (either by actually doing them or mentally walking through steps such as adding new module, removing module, adding and running tests separately in the shared module, IDE integration.. It all seems to hold very well and I could not find any "holes" - but if you do find some, I am happy to look at those as well. What I really like is how eventually "standard" it is... Those shared modules are now .... potentiall distributions that we can use "as usua" ... So what we came up with together is an effective way of building common libraries that can be either "dynamically" (if added as requirement) or "statically" (if symlinked from the same workspace and pre-commit-synced for deps) linked with our code (taking the C libraries analogy). It's surprising how easy it is with workspaces of `uv` and if we are going to proceed with it and find it useful, I am going to pitch it to Astral as a native feature of workspaces (seems that this should be easily generic-able) - following up the https://github.com/astral-sh/uv/issues/14445 -- 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]
