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.

Sign up for GitHub

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

Jump to bottom

Fix nightly clippy lints #1750

Merged
merged 2 commits into from
Feb 17, 2021
Merged

Fix nightly clippy lints #1750

merged 2 commits into from
Feb 17, 2021

Conversation

teor2345
Copy link
Contributor

Motivation

Some of us build with nightly clippy, and like to have warning-free builds.

Here are the latest fixes:

  • capitalised acronyms (rust style is to titlecase acronyms)
  • redundant range

@teor2345 teor2345 added A-rust Area: Updates to Rust code C-cleanup Category: This is a cleanup labels Feb 16, 2021
@teor2345 teor2345 added this to the 2021 Sprint 3 milestone Feb 16, 2021
@teor2345 teor2345 requested a review from a team February 16, 2021 07:26
@teor2345 teor2345 self-assigned this Feb 16, 2021
@teor2345
Copy link
Contributor Author

Yet another tiny routine fix that can just go in this sprint

@teor2345 teor2345 mentioned this pull request Feb 16, 2021
Merged
20 tasks
@teor2345
Copy link
Contributor Author

The CI failure is #1730.

@teor2345 teor2345 merged commit 55613db into ZcashFoundation:main Feb 17, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dconnolly dconnolly dconnolly approved these changes

Assignees

@teor2345 teor2345

Labels
A-rust Area: Updates to Rust code C-cleanup Category: This is a cleanup
Projects
None yet
Milestone
2021 Sprint 3
Development

Successfully merging this pull request may close these issues.
2 participants
@teor2345 @dconnolly