Fury0508 opened a new pull request, #60728:
URL: https://github.com/apache/airflow/pull/60728

   **What change does this PR introduce?**
   
   Strips dev dependencies before running provider YAML validation checks by 
running `uv sync --no-dev --all-packages`.
   
   **Why is this change needed?**
   
   Currently, the provider YAML check runs with all dependencies installed, 
including dev dependencies. This can mask issues where provider code imports 
cross-provider dependencies without proper optional handling (try/except 
blocks). By stripping dev dependencies first, I create an environment closer to 
production and can catch these issues during CI.
   
   **Related issue(s)**
   
   issue #60662
   
   **Changes**
   
   - Added `sync_dependencies_without_dev()` function that runs `uv sync 
--no-dev --all-packages` before provider checks
   - Moved `jsonpath_ng` import from top-level to inside 
`check_providers_are_mentioned_in_issue_template()` as a lazy import (since 
`jsonpath_ng` is a dev dependency, the top-level import would fail after 
stripping dev dependencies)
   
   **Testing**
   
   - Function runs `uv sync --no-dev --all-packages --no-python-downloads 
--no-managed-python`
   - Prints status messages for success/failure
   - Script continues to work after dev dependencies are removed due to lazy 
import fix
   


-- 
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]

Reply via email to