Skip to content

Minor cleanups #81

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
Aug 2, 2025
Merged

Minor cleanups #81

merged 3 commits into from
Aug 2, 2025

Conversation

delan
Copy link
Contributor

@delan delan commented Aug 2, 2025

this patch removes an unnecessary use of Option in Cmds by calling take() on the whole Vec, instead of on each Cmd.

since this repo appears to be formatted with cargo fmt --all with the exception of a single import in examples/dd_test.rs, this patch also formats that import and creates rustfmt.toml as a hint that auto-formatting is ok. that way, i can work on this locally and make commits without having to undo this import change over and over.

i’ve also removed some dead code in StderrThread::new(), which i forgot to remove in #79.

@tao-guo tao-guo merged commit 2eb19ae into rust-shell-script:master Aug 2, 2025
1 check 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