Skip to content

Fix identifying symlinks#378

Merged
j178 merged 1 commit intoj178:masterfrom
branchv:symlinks
Aug 7, 2025
Merged

Fix identifying symlinks#378
j178 merged 1 commit intoj178:masterfrom
branchv:symlinks

Conversation

@branchv
Copy link
Contributor

@branchv branchv commented Aug 7, 2025

Hey, cool project!

When trying to replace pre-commit in my repo, I noticed symlinks weren't being identified since std::fs::metadata resolves symlinks:

This function will traverse symbolic links to query information about the destination file.

@j178
Copy link
Owner

j178 commented Aug 7, 2025

@branchv
Copy link
Contributor Author

branchv commented Aug 7, 2025

Yea makes sense, done

@j178 j178 added the bug Something isn't working label Aug 7, 2025
@j178 j178 merged commit 90646ba into j178:master Aug 7, 2025
15 checks passed
@branchv branchv deleted the symlinks branch August 7, 2025 04:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants