Skip to content

Offer pre-commit hooks - #12

Merged
tauanbinato merged 1 commit into
mainfrom
pre-commit-hooks
Sep 25, 2026
Merged

tauanbinato merged 1 commit into
mainfrom
pre-commit-hooks

Conversation

@tauanbinato

Copy link
Copy Markdown
Contributor

What and why

Phase 3 of the roadmap: .pre-commit-hooks.yaml with two hooks that review the staged changes (jevgate check --base HEAD; pre-commit stashes unstaged changes first).

  • jevgate-system runs the jevgate on PATH (Homebrew, install.sh, cargo).
  • jevgate has pre-commit build JevGate from this repository with Rust.

The README example pins rev: v0.18.0, the first release that has the file.

How it was checked

From a throwaway repository with a staged Python file, pre-commit run passed both hooks with args: [--dry-run]: the Rust hook built JevGate from this commit, the system hook used a local build, and the dry run selected the staged file.

jevgate-system reviews the staged changes with the jevgate on PATH; jevgate builds it from source first. Both run check --base HEAD.
@tauanbinato
tauanbinato merged commit 5763096 into main Sep 25, 2026
7 checks passed
@tauanbinato
tauanbinato deleted the pre-commit-hooks branch September 25, 2026 21:59
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.

1 participant