Release/v2.0.3#40
Conversation
…sibling library and CLI
…d installation instructions
There was a problem hiding this comment.
Pull request overview
This release bumps the version from 2.0.2 to 2.0.3, focusing on documentation improvements, API refinements, and enhanced logging throughout the codebase.
- Comprehensive documentation added with new README files for both the library and CLI
- Error type renamed from
SiblingErrortoErrorfor better naming consistency - Extensive logging support integrated using the
logcrate for debugging and monitoring - API refinements to the
Nextertrait with newnext()andnext_with()method structure
Reviewed changes
Copilot reviewed 10 out of 10 changed files in this pull request and generated 9 comments.
Show a summary per file
| File | Description |
|---|---|
| lib/src/lib.rs | Added extensive documentation comments, integrated logging throughout, renamed SiblingError to Error, refactored Nexter trait API to use next_with() as primary method |
| lib/README.md | New comprehensive README documenting library features, usage examples, API strategies, and error types |
| lib/Cargo.toml | Added log dependency, updated package metadata with additional keywords and categories, added license-file workspace reference |
| cli/src/main.rs | Updated references from SiblingError to Error type |
| cli/src/init.rs | Updated references from SiblingError to Error type |
| cli/README.md | New comprehensive CLI documentation with installation instructions, usage examples, and command reference |
| cli/Cargo.toml | Updated toml dependency from 0.8.19 to 0.9.8, added explicit [[bin]] section, enhanced metadata |
| docs/config.toml | Version string updated from 2.0.2 to 2.0.3 |
| README.md | Removed outdated Requirements section listing specific dependency versions |
| Cargo.toml | Workspace version bumped from 2.0.2 to 2.0.3 |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
…es in printer and lib modules
…CLI and implement Display for Error type
… files for basic directory structure
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 16 out of 45 changed files in this pull request and generated 5 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| echo $result | while IFS=, read current next ci ni total | ||
| do | ||
| # echo "Current: $current, Next: $next, ci: $ci, ni: $ni, total: $total" | ||
| if [[ $sibling_status -eq 0 ]] ; then | ||
| # strip the first and last double quotes | ||
| cd "$(echo $next | sed -e 's/^"//' -e 's/"$//')" | ||
| echo "$PWD ($ci/$total)" | ||
| else | ||
| echo "Done ($ci/$total)" | ||
| cd .. | ||
| fi | ||
| done |
There was a problem hiding this comment.
The cd command inside the while loop runs in a subshell due to the pipe, which means the directory change won't affect the parent shell. This will cause the function to not actually change directories as intended. The solution is to avoid piping into the while loop and instead use process substitution or read the CSV differently.
No description provided.