Fix clippy warnings - Githubissues

nabijaczleweli / cargo-update

A cargo subcommand for checking and applying updates to installed executables

MIT License

1.22k stars 42 forks source link

Fix clippy warnings #29

Closed ishitatsuyuki closed 7 years ago

ishitatsuyuki commented 7 years ago

Also, changed try! to ? operator, and removed some nonsense occurrences.

... please learn about style.

ishitatsuyuki commented 7 years ago

I removed the change to ops::intersect_packages which makes it take a slice instead of Vec, since that will break API compatibility. Do it at your risk and bump the version if doing so.

nabijaczleweli commented 7 years ago

Also rebase upon current master while you're at it.

ishitatsuyuki commented 7 years ago

Reverted try! changes as mentioned. Also did a rebase, although I don't see there's any point to do that (no merge conflicts, and you seems to use the rebase merge button).

nabijaczleweli commented 7 years ago

I don't use buttons.

nabijaczleweli commented 7 years ago

Released in v0.8.2.