Skip to content

release/verify: install pre-commit hooks before pytest to unblock failing run - #1

Draft
Harshith029 with Copilot wants to merge 2 commits into
mainfrom
copilot/fix-verify-github-actions-job
Draft

release/verify: install pre-commit hooks before pytest to unblock failing run#1
Harshith029 with Copilot wants to merge 2 commits into
mainfrom
copilot/fix-verify-github-actions-job

Conversation

Copilot AI commented Aug 7, 2026

Copy link
Copy Markdown

The release workflow’s verify job was failing because it ran tests that assert pre-commit hook installation without first installing those hooks. This PR aligns release verify setup with the existing CI path so the hook-dependent Phase 0 check can execute in the expected environment.

  • Root cause

    • tests/test_phase0_gitleaks.py::test_pre_commit_hook_is_installed expects .git/hooks/pre-commit to exist.
    • release.yml verify job installed dependencies but skipped pre-commit install.
  • Workflow change

    • Added a dedicated pre-commit setup step in jobs.verify of .github/workflows/release.yml:
      • pre-commit install
      • pre-commit install-hooks
  • Effect on verify semantics

    • Keeps lint/type/test gates unchanged.
    • Ensures the verify environment satisfies the repository’s Phase 0 gitleaks/hook contract.
- name: Install pre-commit hooks (gitleaks binary)
  run: |
    pre-commit install
    pre-commit install-hooks

Copilot AI changed the title [WIP] Fix the failing GitHub Actions job verify release/verify: install pre-commit hooks before pytest to unblock failing run Aug 7, 2026
Copilot AI requested a review from Harshith029 August 7, 2026 17:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants