typos icon indicating copy to clipboard operation
typos copied to clipboard

typos-src hook does not run

Open lmmx opened this issue 3 months ago • 3 comments

Description

  • via https://github.com/j178/prek/pull/989#issuecomment-3592522781

Hey there, I was testing out various pre-commit language: rust hooks for prek recently and couldn’t get the typos-src one in this repo’s hooks manifest to work. I don’t know if it was just a draft or if it’s meant to work, but I thought I should mention it in case you weren’t aware :-) I tried it on pre-commit and our new implementation in prek, neither would execute it.

prek itself uses the typos hook, but not the Rust hook!

lmmx avatar Dec 02 '25 00:12 lmmx

Ah, the existing issue isn't in this repo but crate-ci/committed#453.

pre-commit only supports language: rust in a repo root but not all projects are structured that way, particularly if you have a virtual workspace. I switched to a virtual workspace due to issues I had in the past with maturin.

epage avatar Dec 02 '25 13:12 epage

I opened j178/prek#1174 to see if something can be done on prek's side.

epage avatar Dec 02 '25 13:12 epage

Thanks Ed! Yes I can help triage in prek repo :-)

lmmx avatar Dec 02 '25 17:12 lmmx