Skip to content

Conversation

Byron
Copy link
Contributor

@Byron Byron commented Oct 2, 2025

That way it's possible to get the file-id of symlinks, and fully replicate the capabilities of the standard library, but without requiring nightly.

Please not that the majority of the PR was created by Copilot, but I reviewed it thoroughly and cleaned it up just a little.

Related to gitbutlerapp/gitbutler#10528

@Byron Byron marked this pull request as ready for review October 2, 2025 08:12
@JohnTitor
Copy link
Member

Overall LGTM but I'd like to see CI results, could you fix formatting?

…res variants.

That way it's possible to get the file-id of symlinks, and fully replicate the
capabilities of the standard library, but without requiring nightly.
@Byron
Copy link
Contributor Author

Byron commented Oct 4, 2025

Thanks a lot, and I am very sorry for the formatting issue. I did expect my IDE to auto-format it, but apparently that's disabled by default so nothing was formatted.

This I now fixed, and I hope CI will go through without a hitch.

Copy link
Member

@JohnTitor JohnTitor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No problem, thank you!

@JohnTitor JohnTitor merged commit e5f33f7 into notify-rs:main Oct 4, 2025
17 checks passed
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.

2 participants