Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Test graphite #639

Closed
wants to merge 1 commit into from

Add activity feed API

85efaa9
Select commit
Loading
Failed to load commit list.
Sign in for the full log view
Closed

Test graphite #639

Add activity feed API
85efaa9
Select commit
Loading
Failed to load commit list.
GitHub Actions / ffi succeeded Apr 9, 2024

ffi

1 warning

Details

Results

Message level Amount
Internal compiler error 0
Error 0
Warning 1
Note 0
Help 0

Versions

  • rustc 1.77.1 (7cf61ebde 2024-03-27)
  • cargo 1.77.1 (e52e36006 2024-03-26)
  • clippy 0.1.77 (7cf61eb 2024-03-27)

Annotations

Check warning on line 58 in src/mls.rs

See this annotation in the file changed.

@github-actions github-actions / ffi

this function has too many arguments (8/7)

warning: this function has too many arguments (8/7)
  --> src/mls.rs:49:1
   |
49 | / pub async fn create_client(
50 | |     logger: Box<dyn FfiLogger>,
51 | |     host: String,
52 | |     is_secure: bool,
...  |
57 | |     legacy_signed_private_key_proto: Option<Vec<u8>>,
58 | | ) -> Result<Arc<FfiXmtpClient>, GenericError> {
   | |_____________________________________________^
   |
   = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#too_many_arguments
   = note: `#[warn(clippy::too_many_arguments)]` on by default