Skip to content

Apply 1.60 clippy lints and apply variable name - #544

Merged
Defman merged 2 commits into
feather-rs:mainfrom
FussballAndy:main
Jun 17, 2022
Merged

Defman merged 2 commits into
feather-rs:mainfrom
FussballAndy:main

Conversation

@FussballAndy

Copy link
Copy Markdown
Contributor

Status

  • Ready
  • Development
  • Hold

Description

Fixed all new lints that came with clippy 1.60 (and also some of nightly clippy).

Checklist

  • Ran cargo fmt, cargo clippy --all-targets, cargo build --release and cargo test and fixed any generated errors!
  • Removed unnecessary commented out code
  • Used specific traces (if you trace actions please specify the cause i.e. the player)

Note: if you locally don't get any errors, but GitHub Actions fails (especially at clippy) you might want to check your rust toolchain version. You can then feel free to fix these warnings/errors in your PR.

@Defman
Defman merged commit 2f99d76 into feather-rs:main Jun 17, 2022
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.

2 participants