Skip to content
This repository was archived by the owner on Jan 22, 2025. It is now read-only.

fix clippy error #33317

Merged
merged 2 commits into from
Sep 20, 2023
Merged

fix clippy error #33317

merged 2 commits into from
Sep 20, 2023

Conversation

jeffwashington
Copy link
Contributor

@jeffwashington jeffwashington commented Sep 20, 2023 •
edited
Loading

Problem

https://buildkite.com/solana-labs/solana/builds/101939#018aaf88-dadc-4ea1-a104-633b04ee26d1



error: lint `clippy::integer_arithmetic` has been renamed to `clippy::arithmetic_side_effects`
--
  | --> vote/src/lib.rs:2:10
  | \|
  | 2 \| #![allow(clippy::integer_arithmetic)]
  | \|          ^^^^^^^^^^^^^^^^^^^^^^^^^^ help: use the new name: `clippy::arithmetic_side_effects`

Summary of Changes

change clippy allow

Fixes #

@jeffwashington jeffwashington marked this pull request as ready for review September 20, 2023 02:30
pgarg66
pgarg66 previously approved these changes Sep 20, 2023
View reviewed changes
@jeffwashington
Copy link
Contributor Author

source:
rust-lang/rust-clippy#10674

@jeffwashington jeffwashington merged commit 7a8a492 into solana-labs:master Sep 20, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@pgarg66 pgarg66 pgarg66 approved these changes

@t-nelson t-nelson Awaiting requested review from t-nelson

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jeffwashington @pgarg66