Feat/py03 build #33
d-perlAnnotations
16 warnings
|
Rustfmt
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
|
|
Rustfmt
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
|
|
Rustfmt
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
|
|
Rustfmt
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
|
|
Check
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
|
|
Check
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
|
|
Check
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
|
|
Check
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
|
|
function call inside of `expect`:
src/lib/lib.rs#L231
warning: function call inside of `expect`
--> src/lib/redis_logs.rs:231:15
|
231 | create_id.expect(&format!(
| _______________^
232 | | "Failed to create Redis consumer ID {} in group {}!",
233 | | &config.consumer_id, &config.consumer_group
234 | | ));
| |______^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#expect_fun_call
= note: `#[warn(clippy::expect_fun_call)]` on by default
help: try
|
231 ~ create_id.unwrap_or_else(|_| panic!("Failed to create Redis consumer ID {} in group {}!",
232 ~ &config.consumer_id, &config.consumer_group));
|
|
|
called `iter().cloned().collect()` on a slice to create a `Vec`. Calling `to_vec()` is both faster and more readable:
src/lib/lib.rs#L187
warning: called `iter().cloned().collect()` on a slice to create a `Vec`. Calling `to_vec()` is both faster and more readable
--> src/lib/redis_logs.rs:187:48
|
187 | redis::Value::BulkString(x) => Ok(x.iter().cloned().collect()),
| ^^^^^^^^^^^^^^^^^^^^^^^^^^ help: try: `.to_vec()`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#iter_cloned_collect
= note: `#[warn(clippy::iter_cloned_collect)]` on by default
|
|
accessing first element with `raw_reply
.keys.get(0)`:
src/lib/lib.rs#L168
warning: accessing first element with `raw_reply
.keys.get(0)`
--> src/lib/redis_logs.rs:168:19
|
168 | let log_key = raw_reply
| ___________________^
169 | | .keys
170 | | .get(0)
| |_______________^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#get_first
= note: `#[warn(clippy::get_first)]` on by default
help: try
|
168 ~ let log_key = raw_reply
169 + .keys.first()
|
|
|
passing a unit value to a function:
src/lib/lib.rs#L136
warning: passing a unit value to a function
--> src/lib/redis_logs.rs:136:27
|
136 | metadata: {}.into(),
| --^^^^^^^
| |
| help: use a unit literal instead: `()`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#unit_arg
|
|
passing a unit value to a function:
src/lib/lib.rs#L105
warning: passing a unit value to a function
--> src/lib/redis_logs.rs:105:32
|
105 | extra: {}.into(),
| --^^^^^^^
| |
| help: use a unit literal instead: `()`
|
= 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
|
|
use of `unwrap_or` to construct default value:
src/lib/lib.rs#L62
warning: use of `unwrap_or` to construct default value
--> src/lib/elastic_push.rs:62:44
|
62 | let body = make_json_body(&buffer).unwrap_or(vec![]);
| ^^^^^^^^^^^^^^^^^ help: try: `unwrap_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
|
|
redundant closure:
src/lib/lib.rs#L41
warning: redundant closure
--> src/lib/elastic_push.rs:41:14
|
41 | .map(|e| json_from_logmsg(e))
| ^^^^^^^^^^^^^^^^^^^^^^^ help: replace the closure with the function itself: `json_from_logmsg`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#redundant_closure
= note: `#[warn(clippy::redundant_closure)]` on by default
|
|
writing `&Vec` instead of `&[_]` involves a new object where a slice will do:
src/lib/lib.rs#L35
warning: writing `&Vec` instead of `&[_]` involves a new object where a slice will do
--> src/lib/elastic_push.rs:35:11
|
35 | msgs: &Vec<LogMsg>,
| ^^^^^^^^^^^^ help: change this to: `&[LogMsg]`
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#ptr_arg
= note: `#[warn(clippy::ptr_arg)]` on by default
|