Skip to content

test: satisfy clippy::assert_is_empty (Rust 1.99) - #146

Closed
devfive-relay-c70423[bot] wants to merge 1 commit into
mainfrom
devfive/3099154426bffbeef8b62999b80bb038/clippy-assert-is-empty
Closed

devfive-relay-c70423[bot] wants to merge 1 commit into
mainfrom
devfive/3099154426bffbeef8b62999b80bb038/clippy-assert-is-empty

Conversation

@devfive-relay-c70423

Copy link
Copy Markdown

Rust 1.99 added clippy::assert_is_empty (pedantic, enabled here through pedantic = warn and -D warnings). CI now runs 1.99.1, so every PR fails Lint on 34 assert!(x.is_empty()) sites in tests, including docs-only PRs such as #145.

Each site keeps its assert! and gains a failure message, which the lint accepts. No behavior change.

Verified locally with 1.99.1: cargo clippy --all-targets --all-features -- -D warnings clean, cargo fmt --check clean, cargo test --all-features passes.


Submitted via Devfive hub. Employee: ; device: 3099154426bffbeef8b62999b80bb038

@github-actions

Copy link
Copy Markdown

Changepacks

[email protected] → 0.4.1 - Cargo.toml

Patch

  • Test assertions satisfy clippy's assert_is_empty lint (Rust 1.99)

[email protected] - crates/vespera_core/Cargo.toml

Maybe you forgot to write the following files to the latest version

[email protected] - crates/vespera_inprocess/Cargo.toml

Maybe you forgot to write the following files to the latest version

[email protected] - crates/vespera_macro/Cargo.toml

Maybe you forgot to write the following files to the latest version

[email protected] → 0.4.1 - libs/vespera-bridge/build.gradle.kts

Patch

  • Test assertions satisfy clippy's assert_is_empty lint (Rust 1.99)

[email protected] → 0.4.1 - libs/vespera-bridge-gradle-plugin/build.gradle.kts

Patch

  • Test assertions satisfy clippy's assert_is_empty lint (Rust 1.99)

@owjs3901 owjs3901 closed this Oct 31, 2026
@owjs3901
owjs3901 deleted the devfive/3099154426bffbeef8b62999b80bb038/clippy-assert-is-empty branch October 31, 2026 18:10
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.

1 participant