Clippy

# Run clippy on the current crate
cargo clippy

# Automatically apply suggestions
cargo clippy --fix

# Run on all targets (bins, tests, examples)
cargo clippy --all-targets

# Run with all features enabled
cargo clippy --all-features

# Run on a specific package in a workspace
cargo clippy -p example

# Run only on the specified package, excluding dependencies
cargo clippy -p example -- --no-deps

clippy-driver --edition 2021 -Cpanic=abort foo.rs

// Allow a specific lint for the entire crate
#![allow(clippy::single_match)]

// Warn on a lint
#![warn(clippy::pedantic)]

// Deny a lint (makes it an error)
#![deny(clippy::unwrap_used)]

// Allow for a specific function
#[allow(clippy::too_many_arguments)]
fn complex_function(a: i32, b: i32, c: i32, d: i32, e: i32, f: i32, g: i32) {
    // ...
}

# Allow a specific lint
cargo clippy -- -A clippy::lint_name

# Warn on a specific lint
cargo clippy -- -W clippy::lint_name

# Deny all warnings
cargo clippy -- -D warnings

# Enable pedantic lints
cargo clippy -- -W clippy::pedantic

# clippy.toml
avoid-breaking-exported-api = false
disallowed-names = ["toto", "tata", "titi"]

# Extend default values with ".."
disallowed-names = ["bar", ".."]

# clippy.toml
msrv = "1.65.0"

[package]
rust-version = "1.65"

#![feature(custom_inner_attributes)]
#![clippy::msrv = "1.65.0"]

[package]
name = "clippy"
version = "0.1.95"
description = "A bunch of helpful lints to avoid common pitfalls in Rust"
edition = "2024"

[[bin]]
name = "cargo-clippy"
path = "src/main.rs"

[[bin]]
name = "clippy-driver"
path = "src/driver.rs"

[dependencies]
clippy_lints = { path = "clippy_lints" }
clippy_utils = { path = "clippy_utils" }

# Build clippy
./x build clippy

# Test clippy
./x test clippy

# Build and test a specific lint
./x test tests/ui/my_lint.rs

name: Clippy
on: [push, pull_request]

jobs:
  clippy:
    runs-on: ubuntu-latest
    steps:
      - uses: actions/checkout@v4
      - uses: actions-rs/toolchain@v1
        with:
          toolchain: stable
          components: clippy
          override: true
      - run: cargo clippy --all-targets --all-features -- -D warnings

language: rust
rust:
  - stable
  - beta
before_script:
  - rustup component add clippy
script:
  - cargo clippy -- -D warnings
  - cargo test

// ❌ Bad: Incorrect use of clone
let x = Box::new(5);
let y = x.clone(); // Use *x instead

// ❌ Bad: Suspicious else formatting
if condition {
    do_something();
}
else {  // Should be on same line as closing brace
    do_other();
}

// ❌ Bad: Unnecessary collect
let sum: i32 = vec.iter().collect::<Vec<_>>().iter().sum();

// ✅ Good: Direct iteration
let sum: i32 = vec.iter().sum();

// ❌ Bad: Using clone in a loop
for item in items.iter() {
    let owned = item.clone();  // Consider using references
}

// ❌ Bad: Redundant field names
let point = Point { x: x, y: y };

// ✅ Good: Field init shorthand
let point = Point { x, y };

// ❌ Bad: Explicit return
fn add(a: i32, b: i32) -> i32 {
    return a + b;
}

// ✅ Good: Implicit return
fn add(a: i32, b: i32) -> i32 {
    a + b
}

export CLIPPY_DISABLE_DOCS_LINKS=1