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

   This change adds a call to `uv sync --no-dev --all-packages` before running 
the provider YAML validation checks. 
   
   ### Why is this needed?
   
   Currently, the 
[run_provider_yaml_files_check.py](cci:7://file:///Users/dhanyabad/open-source/apache-airflow/airflow/scripts/in_container/run_provider_yaml_files_check.py:0:0-0:0)
 script runs with all development dependencies installed, which may mask issues 
where:
   - Provider code requires dependencies that should be optional
   - Cross-provider dependencies are not properly marked as optional
   
   By running `uv sync --no-dev` before validation, we remove dev dependencies 
and can detect cases where provider code has unhandled optional dependencies.
   
   ### What this PR does:
   - Adds a 
[remove_dev_dependencies()](cci:1://file:///Users/dhanyabad/open-source/apache-airflow/airflow/scripts/in_container/run_provider_yaml_files_check.py:720:0-738:68)
 function that runs `uv sync --no-dev --all-packages`
   - Calls this function at the start of the main block, before 
`ProvidersManager().initialize_providers_configuration()`
   
   Fixes: #60662


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