search

mgunyho / tere

Terminal file explorer
European Union Public License 1.2
1.68k stars 38 forks source link

Apply clippy suggestions #72

Closed orhun closed 2 years ago

orhun commented 2 years ago

This PR updates codebase for fixing the clippy lints. I simply ran cargo clippy --fix

mgunyho commented 2 years ago

Thanks! Could you please re-do this on top of develop, at least one fix has already been applied there.

Interestingly, most of the changes seem related to tests, which are not reported by cargo clippy, but cargo clippy --fix still fixes them.

orhun commented 2 years ago

Thanks! Could you please re-do this on top of develop, at least one fix has already been applied there.

It means that we need to close this PR and I should provide another one for develop I guess 🤔

mgunyho commented 2 years ago

Another option is to rebase, fix possible conflicts, and force-push, either way is fine for me.