Skip to content

Conversation

jamesbraza
Copy link
Owner

This PR moves to the Rust port of pre-commit called prek`:

  • Documents pre-commit and prek in the README tables
  • Adjusts test ordering since prek is so fast it exposed a race condition with pylint
    1. prek would install and run, autofixing pre_tools.py
    2. pylint would trigger different error contents (e.g. different lines)
  • Moves dependencies and CI to use prek

@jamesbraza jamesbraza requested a review from Copilot October 13, 2025 19:29
@jamesbraza jamesbraza self-assigned this Oct 13, 2025
@jamesbraza jamesbraza added the enhancement New feature or request label Oct 13, 2025
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR migrates from pre-commit to prek, a Rust port of pre-commit that provides faster execution and backwards compatibility.

  • Replaces pre-commit dependency with prek in development dependencies
  • Updates documentation to include both tools with detailed comparison table
  • Separates filesystem-mutating tests to run sequentially, avoiding race conditions exposed by prek's speed

Reviewed Changes

Copilot reviewed 4 out of 5 changed files in this pull request and generated 1 comment.

File Description
tests/test_tool_corrections.py Updates test references from pre-commit to prek and adds filesystem marker
pyproject.toml Replaces pre-commit dependency with prek and adds filesystem test marker
README.md Adds comprehensive Git Hooks section documenting both pre-commit and prek
.github/workflows/lint-test.yaml Separates filesystem tests to run sequentially to avoid race conditions

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@jamesbraza jamesbraza merged commit eab8c3a into main Oct 13, 2025
2 checks passed
@jamesbraza jamesbraza deleted the prek branch October 13, 2025 19:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant