Skip to content

Conversation

@gap-editor
Copy link
Contributor

My PR fixes a small inconsistency in the fs utilities module:

Fixed: the FsPathError::ReadLink message was identical to Read ("failed to read from …"), which made it hard to distinguish link read failures in logs.
Now it clearly says “failed to read link …”.
Added: the missing read_link() wrapper for std::fs::read_link, providing consistent API coverage and error context like other helpers (read(), write(), read_dir()).

Copy link
Collaborator

@mattsse mattsse left a comment

Choose a reason for hiding this comment

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

this seems appropriate even tho we don't use it, just in case we want to extract all of this at some point

@github-project-automation github-project-automation bot moved this from Backlog to In Progress in Reth Tracker Oct 27, 2025
@mattsse mattsse merged commit 4f660da into paradigmxyz:main Oct 27, 2025
41 of 42 checks passed
@github-project-automation github-project-automation bot moved this from In Progress to Done in Reth Tracker Oct 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants