fix: skip .git worktree pointer files during walk#80
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: b93ce324f7
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| t.Cleanup(func() { _ = os.Chdir(prev) }) | ||
| } | ||
|
|
||
|
|
There was a problem hiding this comment.
Remove extra blank line to satisfy gofmt
With this inserted blank line, gofmt -l find_replace.go find_replace_test.go prints find_replace_test.go; the repository instructions require gofmt -l . to print nothing before a PR, so this can fail the formatting check even though the code builds otherwise.
Useful? React with 👍 / 👎.
Ignore any entry named .git whether it is a directory or a worktree/ submodule pointer file so find-replace does not corrupt git metadata. Fixes dolph#19
b93ce32 to
ee3d21e
Compare
Fixes #19
Made with Cursor