The GitHub Actions job "CI" on tvm-ffi.git/main has succeeded. Run started by GitHub user tqchen (triggered by tqchen).
Head commit for run: b7cb28b51fbb05a5cd5b1c8a4c9407005e6fe151 / Junru Shao <[email protected]> ci(lint): run hooks through the uv development environment (#669) ## Summary - Convert the pre-commit configuration to local system hooks whose executables come from the uv development dependency group. - Pin the hook toolchain in `pyproject.toml`, align the Ruff D102 test exemption, and remove redundant per-file suppressions. - Run pre-commit explicitly through uv in GitHub Actions so CI uses the environment provisioned by `uv sync`. ## Architecture - The uv `dev` dependency group is the single source of truth for hook tool versions. - Pre-commit remains the orchestration layer, while hook processes execute from the active project environment instead of isolated repository environments. - The CI lint job now provisions and consumes the same environment, eliminating the runtime boundary that caused `pre-commit/action` to miss the pinned executables. ## Public Interfaces - No runtime API, ABI, schema, or CLI behavior changes. - The development dependency group now requires Python 3.10 or newer. ## UI/UX - None. ## Behavioral Changes - Ruff, ty, clang-format, YAML, Cython, shell, and CMake checks resolve from the uv development environment. - CI invokes `uv run --no-sync pre-commit run --show-diff-on-failure --color=always --all-files` after provisioning that environment. - The D102 exemption is applied centrally, and redundant test-file suppressions are removed. - Hooks not supplied by the development environment are no longer configured. ## Breaking Changes and Migration - No runtime breaking changes. - Contributors running hooks outside an activated development environment should use `uv run --no-sync pre-commit ...` after syncing the `dev` dependency group. ## Docs - No user-facing documentation changes are required; the development and CI configuration directly records the workflow. ## Tests - `uv run --no-sync pre-commit validate-config .pre-commit-config.yaml` - `uv run --no-sync ruff check tests/python/test_dataclass_common.py tests/python/test_dataclass_copy.py tests/python/test_dataclass_frozen.py tests/python/test_dataclass_init.py tests/python/test_dataclass_py_class.py` - `uv sync --group dev --no-install-project` - `uv run --no-sync pre-commit run --show-diff-on-failure --color=always --all-files` — all configured hooks passed. - `git diff --check` and `git diff --cached --check` — passed. ## Untested Edge Cases - Fresh development-environment provisioning on every supported platform has not been exercised locally. - The GitHub-hosted Ubuntu runner execution is left to CI; risk is low because the exact workflow command passed against the provisioned uv environment. ## Commits - `a385ed3d` — centralize system hooks in the uv development environment. - `338e8b4a` — invoke pre-commit through uv in CI. Report URL: https://github.com/apache/tvm-ffi/actions/runs/29455139041 With regards, GitHub Actions via GitBox --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
