-
Notifications
You must be signed in to change notification settings - Fork 53
Description
I don't have the time/knowledge to debug this fully but hoping you guys might have some insight.
A while back I cloned the-via/app
and must've run some npm build command or something. It wasn't until a few weeks later that I noticed a git hook installed in my home directory (which I have a git repo at), in ~/.git/hooks/pre-commit
. I see that a dependency of the-via/app
, the-via/keyboards
, uses simple-git-hooks
to put the exact hook (npm run prettify-staged
) that I found in my home dir, and the file creation date lines up. So I suspect this is how that hook got created.
Could it have to do with how getProjectRootDirectoryFromNodeModules
walks up the directory tree to try and find the actual root of the project simple-git-hooks
is being run in? I looked at that function and proceeded to check if my home dir has any of .pnpm
, .deno
, .store
, or node_modules
but it does not. It has a .yarn
but it didn't look like that function was relying on that dir name.
If it helps, my home dir is exactly 2 dirs above where I clone repos to.
$ grep -C 3 simple ~/projects/via-app/node_modules/via-keyboards/package.json
"@types/node": "^18.14.0",
"prettier": "^2.8.4",
"pretty-quick": "^3.1.3",
"simple-git-hooks": "^2.8.1"
},
"dependencies": {
"@the-via/reader": "^1.9.0",
--
"ts-node": "10.9.1",
"typescript": "^4.9.5"
},
"simple-git-hooks": {
"pre-commit": "npm run prettify-staged"
}
}
Also, I had a little trouble following what parts of the code in simple-git-hooks
could've been run from the build in the-via/app
, but is it possible that getGitProjectRoot
doesn't work right, since I do have a .git in my home directory? I can't tell if that function is used for the setting hooks logic.