Skip to content

fix: resolve CI failures (clippy + fmt)#10

Merged
terraboops merged 3 commits intomainfrom
fix/ci-clippy-warnings
Feb 4, 2026
Merged

fix: resolve CI failures (clippy + fmt)#10
terraboops merged 3 commits intomainfrom
fix/ci-clippy-warnings

Conversation

@terraboops
Copy link
Owner

Summary

  • Remove unnecessary borrow in stream.rs:1111 (needless_borrows_for_generic_args clippy lint)
  • Add #[allow(dead_code)] to unused test utility functions in xmpp_client.rs:
    • establish_session
    • get_roster
    • is_success
    • is_failure
  • Apply cargo fmt to fix formatting issues across the codebase

Test plan

  • cargo clippy --workspace --all-targets -- -D warnings passes
  • cargo fmt --check passes

- Remove unnecessary borrow in stream.rs:1111 (needless_borrows_for_generic_args)
- Add #[allow(dead_code)] to unused test utility functions in xmpp_client.rs
- Apply cargo fmt to fix formatting issues
Rustls 0.23+ requires explicitly selecting a crypto provider.
Enable the 'ring' feature for cross-platform compatibility.
When both ring and aws-lc-rs features are enabled (via dependency tree),
rustls cannot auto-detect which provider to use. Explicitly install
ring as the default provider before building ServerConfig.
@terraboops terraboops merged commit 6b79dc5 into main Feb 4, 2026
6 checks passed
terraboops added a commit that referenced this pull request Feb 4, 2026
fix: resolve CI failures (clippy + fmt)
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

Comments