aembke / redis-protocol.rs

A Rust implementation of RESP2 and RESP3
Apache License 2.0
40 stars 14 forks source link

Cleanup #4

Closed ckampfe closed 3 years ago

ckampfe commented 5 years ago

Hello! Thanks for the great library, I appreciate it! This PR is some general tidying up. It makes no changes to the functionality at all.

It only does:

Thanks!

aembke commented 5 years ago

hey @ckampfe thanks for the helpful PR and my apologies getting to it so late. we use this at Okta but we have a slightly different style guide so in the interest of consistency i'm likely going to take up the clippy changes and compiler warning changes, but with a different rustfmt config.

ckampfe commented 5 years ago

thanks!

aembke commented 3 years ago

Thanks for filing this. We used different formatting on the team and finally got around to updating this part of the toolchain. To avoid conflicts I'm going to close this now that all of the additions here have been incorporated into the crate.