Skip to content

Add uv sync --no-dev before provider YAML checks #60728

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
Fury0508 wants to merge 6 commits into
base: main
Choose a base branch
from
Open

Add uv sync --no-dev before provider YAML checks #60728

Changes from all commits
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
4dcca10
Add sync_dependencies_without_dev() and call it before provider checks
Fury0508 Jan 18, 2026
7209f09
Add uv sync --no-dev before provider YAML checks
Fury0508 Jan 17, 2026
d56acb5
Change uv sync failure handling from warning to exit
Fury0508 Jan 17, 2026
481a8ae
Add sync_dependencies_without_dev() and call it before provider checks
Fury0508 Jan 18, 2026
625d604
ixes subprocess.run to include explicit check=False
Fury0508 Jan 18, 2026
018fb69
ran perk --all-files
Fury0508 Jan 27, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
27 changes: 27 additions & 0 deletions scripts/in_container/run_provider_yaml_files_check.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -25,6 +25,7 @@
import os
import pathlib
import platform
import subprocess
import sys
import textwrap
import warnings
Expand Down Expand Up @@ -101,6 +102,31 @@
suspended_integrations: set[str] = set()


def sync_dependencies_without_dev() -> None:
"""
Run uv sync --no-dev to strip development dependencies.

This ensures validation runs in an environment closer to production,
which helps detect cases where providers have unhandled optional
cross-provider dependencies.
"""
console.print("[magenta]Running uv sync --no-dev to strip development dependencies...[/]")
result = subprocess.run(
["uv", "sync", "--no-dev", "--all-packages", "--no-python-downloads", "--no-managed-python"],
capture_output=True,
text=True,
cwd=AIRFLOW_ROOT_PATH,
check=False,
)
if result.returncode != 0:
console.print(f"[red]Failed to remove dev dependencies: {result.stderr}[/]")
sys.exit(1)

console.print("[green]Successfully synchronized without dev dependencies[/]")
if result.stdout:
console.print(result.stdout)


def _filepath_to_module(filepath: pathlib.Path | str) -> str:
if isinstance(filepath, str):
filepath = pathlib.Path(filepath)
Expand Down Expand Up @@ -718,6 +744,7 @@ def check_providers_have_all_documentation_files(yaml_files: dict[str, dict]):


if __name__ == "__main__":
sync_dependencies_without_dev()
ProvidersManager().initialize_providers_configuration()
architecture = Architecture.get_current()
console.print(f"Verifying packages on {architecture} architecture. Platform: {platform.machine()}.")
Expand Down