Skip to content

Conversation

@paddyroddy
Copy link
Owner

Issue running it locally

@paddyroddy paddyroddy requested a review from Copilot February 11, 2026 12:38
@paddyroddy paddyroddy self-assigned this Feb 11, 2026
@paddyroddy paddyroddy added bug Something isn't working prek/pre-commit prek/pre-commit labels Feb 11, 2026
Copy link
Contributor

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

Fixes local execution issues for prek hooks by switching hook config paths to absolute (module-relative) paths.

Changes:

  • Update language hook entrypoints to pass module-relative config paths into run_hooks.
  • Adjust run_hooks to use the provided config path directly (instead of prefixing with a base dir).
  • Pin the shared pre-commit repo to a specific commit.

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 6 comments.

Show a summary per file
File Description
prek/python/python_hooks.py Passes an absolute/module-relative YAML path to run_hooks.
prek/lua/lua_hooks.py Passes an absolute/module-relative YAML path to run_hooks.
prek/latex/latex_hooks.py Passes an absolute/module-relative YAML path to run_hooks.
prek/general/general_hooks.py Passes an absolute/module-relative YAML path to run_hooks.
prek/_run_hooks.py Stops prefixing the config path; uses the given hooks_path as-is.
.pre-commit-config.yaml Switches rev from a tag to a commit SHA.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@paddyroddy paddyroddy merged commit 99019c6 into main Feb 11, 2026
2 of 3 checks passed
@paddyroddy paddyroddy deleted the bug branch February 11, 2026 12:44
@github-project-automation github-project-automation bot moved this from TODO to Done in @paddyroddy Feb 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working prek/pre-commit prek/pre-commit

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant