📝 Fix inaccurate EntryBuilder factory method summaries#2905
Conversation
Also fix misleading path strings in symlink/hard link examples where "path/of/target" was used for the link name and "path/of/source" for the link target, reversing the actual semantics.
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughUpdated Rustdoc comments for Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Possibly related PRs
Suggested labels
Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Code Review
This pull request updates the documentation for EntryBuilder methods, including new_dir, new_file, new_symlink, and new_hard_link, to more accurately describe their return types and parameters. Specifically, it clarifies that these methods return an EntryBuilder and improves the naming in example code to better distinguish between the link path and its target. I have no feedback to provide.
Also fix misleading path strings in symlink/hard link examples where "path/of/target" was used for the link name and "path/of/source" for the link target, reversing the actual semantics.
Summary by CodeRabbit