Skip to content

Add Docker pre-commit hook and version consistency check - #74

Merged
andrewring merged 4 commits into
mainfrom
dev/andrewring/docker-pre-commit-hook
Jun 17, 2026
Merged

Add Docker pre-commit hook and version consistency check#74
andrewring merged 4 commits into
mainfrom
dev/andrewring/docker-pre-commit-hook

Conversation

@andrewring

@andrewring andrewring commented Jun 16, 2026

Copy link
Copy Markdown
Owner

Summary

  • Adds a github-distributed-owners-docker pre-commit hook using language: docker_image, which pulls the pre-built GHCR image instead of compiling from source. No Rust toolchain required — just Docker.
  • The existing language: rust hook is unchanged.
  • Updates README with Docker hook usage example.
  • Adds a version consistency check to the Build & Test workflow that verifies hardcoded versions stay in sync:
    • Package version (Cargo.toml) matches .pre-commit-hooks.yaml docker image tag and README.md rev examples
    • Rust version (rust-toolchain.toml) matches Cargo.toml rust-version and all workflow toolchain references

Test plan

  • Verify version consistency check passes in CI
  • Test Docker pre-commit hook locally: pre-commit try-repo . github-distributed-owners-docker --all-files

Generated with Devin

andrewring and others added 4 commits June 16, 2026 17:55
Adds a github-distributed-owners-docker hook using language:
docker_image, which pulls the pre-built GHCR image instead of
compiling from source. This removes the Rust toolchain requirement
for users who have Docker available.

The existing language: rust hook is unchanged.

Generated with [Devin](https://cli.devin.ai/docs)

Co-Authored-By: Devin <158243242+devin-ai-integration[bot]@users.noreply.github.com>
Verifies that hardcoded versions stay in sync across the repo:
- Package version (Cargo.toml) matches .pre-commit-hooks.yaml
  docker image tag and README rev examples
- Rust version (rust-toolchain.toml) matches Cargo.toml
  rust-version and all workflow toolchain references

Generated with [Devin](https://cli.devin.ai/docs)

Co-Authored-By: Devin <158243242+devin-ai-integration[bot]@users.noreply.github.com>
The tool shells out to git (via `git ls-files`) to filter untracked
files. A scratch image has no git binary, causing "No such file or
directory" errors at runtime. Alpine adds git with minimal overhead.

Generated with [Devin](https://cli.devin.ai/docs)

Co-Authored-By: Devin <158243242+devin-ai-integration[bot]@users.noreply.github.com>
@andrewring
andrewring enabled auto-merge (squash) June 17, 2026 00:39
@andrewring
andrewring disabled auto-merge June 17, 2026 00:45
@andrewring
andrewring merged commit 89ceb39 into main Jun 17, 2026
59 of 60 checks passed
@andrewring
andrewring deleted the dev/andrewring/docker-pre-commit-hook branch June 17, 2026 00:45
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