Skip to content
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

Fix all clippy warnings #422

Merged
merged 1 commit into from Oct 3, 2020
Merged

Conversation

rhysd
Copy link
Contributor

@rhysd rhysd commented Oct 3, 2020

This PR fixes all warnings from cargo clippy and cargo fmt -- --check.

Copy link
Collaborator

@ordian ordian left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

Strange it wasn't caught by the CI.

@ordian ordian merged commit 2a08f03 into killercup:master Oct 3, 2020
ordian added a commit to ordian/cargo-edit that referenced this pull request Oct 3, 2020
* master: (21 commits)
  Fix fetch for non lowercase crate names (killercup#398)
  Fix all clippy warnings (killercup#422)
  Keep dependencies in sorted order if they were already sorted (killercup#421)
  refacored getting checkout name into a function.
  convert to file_name earlier
  Fix to allow any remote name to be used
  Replace use and suggestion of deprecated `--all` (killercup#407)
  Fix pkgid option descrion for `rm` command (killercup#405)
  Normalise windows-style paths (killercup#403)
  "Working with features" updated (killercup#395)
  release 0.6.0 (killercup#393)
  Add `--workspace` to cargo-upgrade and deprecate `--all` (killercup#392)
  Add `--package <pkgid>` to all commands (killercup#390)
  Fix clippy warnings and some errors on Windows (killercup#382)
  tests: Replace tempdir with tempfile (killercup#380)
  Dependency support for git branches (killercup#379)
  Update help output in readme file (killercup#378)
  bump version to 0.5.0 (killercup#376)
  Add a dependency that was renamed previously (killercup#351)
  Upgrade to reqwest 0.10 (killercup#368)
  ...
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.

None yet

2 participants