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

[RSDK-6152] Downgrade build image requirement #88

Merged
merged 17 commits into from
Jan 10, 2024

Clean up runners

8565711
Select commit
Loading
Failed to load commit list.
Sign in for the full log view
Merged

[RSDK-6152] Downgrade build image requirement #88

Clean up runners
8565711
Select commit
Loading
Failed to load commit list.
GitHub Actions / clippy succeeded Jan 10, 2024 in 1s

clippy

30 warnings

Details

Results

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

Versions

  • rustc 1.75.0 (82e1608df 2023-12-21)
  • cargo 1.75.0 (1d8b05cdd 2023-11-20)
  • clippy 0.1.75 (82e1608 2023-12-21)

Annotations

Check warning on line 326 in src/dialdbg/main.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

redundant pattern matching, consider using `is_ok()`

warning: redundant pattern matching, consider using `is_ok()`
   --> src/dialdbg/main.rs:326:16
    |
326 |         if let Ok(_) = log_path.try_exists() {
    |         -------^^^^^------------------------ help: try: `if log_path.try_exists().is_ok()`
    |
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_pattern_matching

Check warning on line 258 in src/dialdbg/main.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

redundant pattern matching, consider using `is_ok()`

warning: redundant pattern matching, consider using `is_ok()`
   --> src/dialdbg/main.rs:258:16
    |
258 |         if let Ok(_) = log_path.try_exists() {
    |         -------^^^^^------------------------ help: try: `if log_path.try_exists().is_ok()`
    |
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_pattern_matching
    = note: `#[warn(clippy::redundant_pattern_matching)]` on by default

Check warning on line 152 in src/dialdbg/main.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

length comparison to zero

warning: length comparison to zero
   --> src/dialdbg/main.rs:152:8
    |
152 |     if responses.len() == 0 {
    |        ^^^^^^^^^^^^^^^^^^^^ help: using `is_empty` is clearer and more explicit: `responses.is_empty()`
    |
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#len_zero

Check warning on line 13 in src/dialdbg/stats.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

you seem to want to iterate on a map's values

warning: you seem to want to iterate on a map's values
  --> src/dialdbg/stats.rs:13:27
   |
13 |         for (_, value) in &self.0.reports {
   |                           ^^^^^^^^^^^^^^^
   |
   = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#for_kv_map
   = note: `#[warn(clippy::for_kv_map)]` on by default
help: use the corresponding method
   |
13 |         for value in self.0.reports.values() {
   |             ~~~~~    ~~~~~~~~~~~~~~~~~~~~~~~

Check warning on line 159 in src/dialdbg/parse.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

length comparison to zero

warning: length comparison to zero
   --> src/dialdbg/parse.rs:159:8
    |
159 |     if split_log.len() == 0 {
    |        ^^^^^^^^^^^^^^^^^^^^ help: using `is_empty` is clearer and more explicit: `split_log.is_empty()`
    |
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#len_zero
    = note: `#[warn(clippy::len_zero)]` on by default

Check warning on line 10 in src/dialdbg/parse.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

constants have by default a `'static` lifetime

warning: constants have by default a `'static` lifetime
  --> src/dialdbg/parse.rs:10:38
   |
10 | pub(crate) const DIAL_ERROR_PREFIX: &'static str = "unexpected dial connect error";
   |                                     -^^^^^^^---- help: consider removing `'static`: `&str`
   |
   = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_static_lifetimes
   = note: `#[warn(clippy::redundant_static_lifetimes)]` on by default

Check warning on line 351 in src/rpc/webrtc.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

you seem to be trying to use `match` for destructuring a single pattern. Consider using `if let`

warning: you seem to be trying to use `match` for destructuring a single pattern. Consider using `if let`
   --> src/rpc/webrtc.rs:344:5
    |
344 | /     match proto.status {
345 | |         Some(ref status) => {
346 | |             let key = HeaderName::from_str("Grpc-Message").unwrap();
347 | |             let val = HeaderValue::from_str(&status.message).unwrap();
...   |
350 | |         None => (),
351 | |     }
    | |_____^
    |
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#single_match
    = note: `#[warn(clippy::single_match)]` on by default
help: try
    |
344 ~     if let Some(ref status) = proto.status {
345 +         let key = HeaderName::from_str("Grpc-Message").unwrap();
346 +         let val = HeaderValue::from_str(&status.message).unwrap();
347 +         trailers.insert(key, val);
348 +     }
    |

Check warning on line 308 in src/rpc/webrtc.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

unneeded `return` statement

warning: unneeded `return` statement
   --> src/rpc/webrtc.rs:308:13
    |
308 |             return Err(anyhow::anyhow!("Action timed out"));
    |             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    |
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return

Check warning on line 305 in src/rpc/webrtc.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

unneeded `return` statement

warning: unneeded `return` statement
   --> src/rpc/webrtc.rs:305:13
    |
305 |             return Ok(res);
    |             ^^^^^^^^^^^^^^
    |
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return
    = note: `#[warn(clippy::needless_return)]` on by default

Check warning on line 608 in src/rpc/dial.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

using `clone` on type `Option<&Authority>` which implements the `Copy` trait

warning: using `clone` on type `Option<&Authority>` which implements the `Copy` trait
   --> src/rpc/dial.rs:608:22
    |
608 |         let domain = original_uri.authority().clone().unwrap().to_string();
    |                      ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: try removing the `clone` call: `original_uri.authority()`
    |
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#clone_on_copy
    = note: `#[warn(clippy::clone_on_copy)]` on by default

Check warning on line 523 in src/rpc/dial.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

this expression creates a reference which is immediately dereferenced by the compiler

warning: this expression creates a reference which is immediately dereferenced by the compiler
   --> src/rpc/dial.rs:523:67
    |
523 |                 Self::create_channel(self.config.allow_downgrade, &domain, uri.clone(), false)
    |                                                                   ^^^^^^^ help: change this to: `domain`
    |
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrow

Check warning on line 505 in src/rpc/dial.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

this expression creates a reference which is immediately dereferenced by the compiler

warning: this expression creates a reference which is immediately dereferenced by the compiler
   --> src/rpc/dial.rs:505:67
    |
505 |                 Self::create_channel(self.config.allow_downgrade, &domain, uri, true).await
    |                                                                   ^^^^^^^ help: change this to: `domain`
    |
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrow

Check warning on line 408 in src/rpc/dial.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

use of `or_insert` to construct default value

warning: use of `or_insert` to construct default value
   --> src/rpc/dial.rs:408:30
    |
408 |                 map.entry(k).or_insert(vec![]).push(v);
    |                              ^^^^^^^^^^^^^^^^^ help: try: `or_default()`
    |
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#unwrap_or_default
    = note: `#[warn(clippy::unwrap_or_default)]` on by default

Check warning on line 114 in src/rpc/client_stream.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

passing a unit value to a function

warning: passing a unit value to a function
   --> src/rpc/client_stream.rs:114:47
    |
114 |             Some(Type::Trailers(trailers)) => Ok(self.process_trailers(trailers.to_owned()).await),
    |                                               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    |
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#unit_arg
    = note: `#[warn(clippy::unit_arg)]` on by default
help: move the expression in front of the call and replace it with the unit literal `()`
    |
114 ~             Some(Type::Trailers(trailers)) => {
115 +                 self.process_trailers(trailers.to_owned()).await;
116 +                 Ok(())
117 ~             },
    |

Check warning on line 275 in src/ffi/spatialmath/quaternion.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

this expression creates a reference which is immediately dereferenced by the compiler

warning: this expression creates a reference which is immediately dereferenced by the compiler
   --> src/ffi/spatialmath/quaternion.rs:275:20
    |
275 |     to_raw_pointer(&quat)
    |                    ^^^^^ help: change this to: `quat`
    |
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrow
    = note: `#[warn(clippy::needless_borrow)]` on by default

Check warning on line 265 in src/ffi/dial_ffi.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

non-binding `let` on a future

warning: non-binding `let` on a future
   --> src/ffi/dial_ffi.rs:263:5
    |
263 | /     let _ = runtime.spawn(async {
264 | |         let _ = server.await;
265 | |     });
    | |_______^
    |
    = help: consider awaiting the future or dropping explicitly with `std::mem::drop`
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#let_underscore_future
    = note: `#[warn(clippy::let_underscore_future)]` on by default

Check warning on line 229 in src/ffi/dial_ffi.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

returning the result of a `let` binding from a block

warning: returning the result of a `let` binding from a block
   --> src/ffi/dial_ffi.rs:229:17
    |
226 | /                 let res = timeout(
227 | |                     timeout_duration, 
228 | |                     dial_without_cred(uri_str, allow_insec, disable_webrtc)?.connect()).await?;
    | |_______________________________________________________________________________________________- unnecessary `let` binding
229 |                   res
    |                   ^^^
    |
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#let_and_return
help: return the expression directly
    |
226 ~                 
227 ~                 timeout(
228 +                     timeout_duration, 
229 +                     dial_without_cred(uri_str, allow_insec, disable_webrtc)?.connect()).await?
    |

Check warning on line 223 in src/ffi/dial_ffi.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

returning the result of a `let` binding from a block

warning: returning the result of a `let` binding from a block
   --> src/ffi/dial_ffi.rs:223:17
    |
210 | /                 let res = timeout(
211 | |                     timeout_duration, 
212 | |                     dial_with_cred(
213 | |                     uri_str,
...   |
221 | |                 )
222 | |                 .await?;
    | |________________________- unnecessary `let` binding
223 |                   res
    |                   ^^^
    |
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#let_and_return
    = note: `#[warn(clippy::let_and_return)]` on by default
help: return the expression directly
    |
210 ~                 
211 ~                 timeout(
212 +                     timeout_duration, 
213 +                     dial_with_cred(
214 +                     uri_str,
215 +                     entity_opt,
216 +                     t.to_str()?,
217 +                     p.to_str()?,
218 +                     allow_insec,
219 +                     disable_webrtc,
220 +                 )?
221 +                 .connect()
222 +                 )
223 +                 .await?
    |

Check warning on line 22 in src/rpc/log_prefixes.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

constants have by default a `'static` lifetime

warning: constants have by default a `'static` lifetime
  --> src/rpc/log_prefixes.rs:22:34
   |
22 | pub const ICE_CONNECTED_EXTERN: &'static str = "ICE connection state changed: connected";
   |                                 -^^^^^^^---- help: consider removing `'static`: `&str`
   |
   = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_static_lifetimes

Check warning on line 18 in src/rpc/log_prefixes.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

constants have by default a `'static` lifetime

warning: constants have by default a `'static` lifetime
  --> src/rpc/log_prefixes.rs:18:32
   |
18 | pub const CANDIDATE_SELECTED: &'static str = "Selected candidate pair";
   |                               -^^^^^^^---- help: consider removing `'static`: `&str`
   |
   = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_static_lifetimes

Check warning on line 16 in src/rpc/log_prefixes.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

constants have by default a `'static` lifetime

warning: constants have by default a `'static` lifetime
  --> src/rpc/log_prefixes.rs:16:27
   |
16 | pub const DIALED_WEBRTC: &'static str = "Connected via WebRTC";
   |                          -^^^^^^^---- help: consider removing `'static`: `&str`
   |
   = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_static_lifetimes

Check warning on line 15 in src/rpc/log_prefixes.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

constants have by default a `'static` lifetime

warning: constants have by default a `'static` lifetime
  --> src/rpc/log_prefixes.rs:15:25
   |
15 | pub const DIALED_GRPC: &'static str = "Connected via gRPC";
   |                        -^^^^^^^---- help: consider removing `'static`: `&str`
   |
   = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_static_lifetimes

Check warning on line 14 in src/rpc/log_prefixes.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

constants have by default a `'static` lifetime

warning: constants have by default a `'static` lifetime
  --> src/rpc/log_prefixes.rs:14:26
   |
14 | pub const DIAL_ATTEMPT: &'static str = "Dialing";
   |                         -^^^^^^^---- help: consider removing `'static`: `&str`
   |
   = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_static_lifetimes

Check warning on line 12 in src/rpc/log_prefixes.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

constants have by default a `'static` lifetime

warning: constants have by default a `'static` lifetime
  --> src/rpc/log_prefixes.rs:12:43
   |
12 | pub const END_LOCAL_SESSION_DESCRIPTION: &'static str = "End local session description";
   |                                          -^^^^^^^---- help: consider removing `'static`: `&str`
   |
   = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_static_lifetimes

Check warning on line 11 in src/rpc/log_prefixes.rs

See this annotation in the file changed.

@github-actions github-actions / clippy

constants have by default a `'static` lifetime

warning: constants have by default a `'static` lifetime
  --> src/rpc/log_prefixes.rs:11:45
   |
11 | pub const START_LOCAL_SESSION_DESCRIPTION: &'static str = "Start local session description";
   |                                            -^^^^^^^---- help: consider removing `'static`: `&str`
   |
   = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_static_lifetimes