Fury0508 commented on PR #60728: URL: https://github.com/apache/airflow/pull/60728#issuecomment-3877405998
@potiuk - You're right that running uv sync at the distribution folder level would be a cleaner approach. I can see how using the workspace feature would automatically sync all provider dependencies. I'll look into the FOSDEM presentation you linked to better understand the implementation. @shahar1 - I see you created #61713 which handles the pre-commit validation side. That makes sense to have both layers: Your PR catches issues at pre-commit stage (faster feedback) This PR ensures we have the full E2E validation when running the actual YAML checks Current status: This PR is approved and all checks are passing However, based on potiuk's suggestion, I think we should refactor this to use the uv sync approach at the provider folder level rather than the current --no-dev flag implementation Some work remains to implement this better approach Sorry for the delay in my work. -- 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]
