Skip to content

fix: unset GIT_ vars to fix pre-commit in worktree - #2293

Merged
mre merged 1 commit into
masterfrom
unset-git-dir
Sep 6, 2026
Merged

mre merged 1 commit into
masterfrom
unset-git-dir

Conversation

@katrinafyi

Copy link
Copy Markdown
Member

Closes #2292

@mre

mre commented Sep 4, 2026

Copy link
Copy Markdown
Member

Hum, any idea why the tests don't pass?

@katrinafyi

Copy link
Copy Markdown
Member Author

Not sure. Seems to be a 429 but does also seem to be consistent. We could try to remove the problematic 10Gb URL.

@mre

mre commented Sep 6, 2026

Copy link
Copy Markdown
Member

Either way, I guess the failure is unrelated to the branch's code, so I'll go ahead and merge it. 😄 Thanks for the fix.

@mre
mre merged commit 81cb43e into master Sep 6, 2026
24 of 29 checks passed
@mre
mre deleted the unset-git-dir branch September 6, 2026 15:21
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.

pre-commit hook fails from a linked git worktree — hook-exported GIT_DIR points the wrapper's git calls at the user's repository

2 participants