[INFO] cloning repository https://github.com/mattmacf98/codecrafters-redis-rust
[INFO] running `Command { std: "git" "-c" "credential.helper=" "-c" "credential.helper=/workspace/cargo-home/bin/git-credential-null" "clone" "--bare" "https://github.com/mattmacf98/codecrafters-redis-rust" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Fmattmacf98%2Fcodecrafters-redis-rust", kill_on_drop: false }`
[INFO] [stderr] Cloning into bare repository '/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Fmattmacf98%2Fcodecrafters-redis-rust'...
[INFO] running `Command { std: "git" "rev-parse" "HEAD", kill_on_drop: false }`
[INFO] [stdout] 16be582b3cfc73edc8d0ac5b660a496d0d2a12ab
[INFO] checking mattmacf98/codecrafters-redis-rust against a5689b034ad287112f709693384a37f1377eb9cc for pr-129249-3
[INFO] running `Command { std: "git" "clone" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Fmattmacf98%2Fcodecrafters-redis-rust" "/workspace/builds/worker-3-tc2/source", kill_on_drop: false }`
[INFO] [stderr] Cloning into '/workspace/builds/worker-3-tc2/source'...
[INFO] [stderr] done.
[INFO] started tweaking git repo https://github.com/mattmacf98/codecrafters-redis-rust
[INFO] finished tweaking git repo https://github.com/mattmacf98/codecrafters-redis-rust
[INFO] tweaked toml for git repo https://github.com/mattmacf98/codecrafters-redis-rust written to /workspace/builds/worker-3-tc2/source/Cargo.toml
[INFO] validating manifest of git repo https://github.com/mattmacf98/codecrafters-redis-rust on toolchain a5689b034ad287112f709693384a37f1377eb9cc
[INFO] running `Command { std: CARGO_HOME="/workspace/cargo-home" RUSTUP_HOME="/workspace/rustup-home" "/workspace/cargo-home/bin/cargo" "+a5689b034ad287112f709693384a37f1377eb9cc" "metadata" "--manifest-path" "Cargo.toml" "--no-deps", kill_on_drop: false }`
[INFO] crate git repo https://github.com/mattmacf98/codecrafters-redis-rust already has a lockfile, it will not be regenerated
[INFO] running `Command { std: CARGO_HOME="/workspace/cargo-home" RUSTUP_HOME="/workspace/rustup-home" "/workspace/cargo-home/bin/cargo" "+a5689b034ad287112f709693384a37f1377eb9cc" "fetch" "--manifest-path" "Cargo.toml", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-3-tc2/source:/opt/rustwide/workdir:ro,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-3-tc2/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/cargo-home:/opt/rustwide/cargo-home:ro,Z" "-v" "/var/lib/crater-agent-workspace/rustup-home:/opt/rustwide/rustup-home:ro,Z" "-m" "1610612736" "--network" "none" "ghcr.io/rust-lang/crates-build-env/linux@sha256:77db811e55d90add9212f6832c23229e3de5f1c1f9905e8cb07319ea15df73ac" "sleep" "infinity", kill_on_drop: false }`
[INFO] [stdout] 9e8c78632efc49d5ca4be88a21d5c6ad019702f3e8f351342d158061c34c78c4
[INFO] running `Command { std: "docker" "start" "9e8c78632efc49d5ca4be88a21d5c6ad019702f3e8f351342d158061c34c78c4", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "inspect" "9e8c78632efc49d5ca4be88a21d5c6ad019702f3e8f351342d158061c34c78c4", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "exec" "-e" "SOURCE_DIR=/opt/rustwide/workdir" "-e" "CARGO_HOME=/opt/rustwide/cargo-home" "-e" "RUSTUP_HOME=/opt/rustwide/rustup-home" "-e" "CARGO_TARGET_DIR=/opt/rustwide/target" "-w" "/opt/rustwide/workdir" "--user" "0:0" "9e8c78632efc49d5ca4be88a21d5c6ad019702f3e8f351342d158061c34c78c4" "/opt/rustwide/cargo-home/bin/cargo" "+a5689b034ad287112f709693384a37f1377eb9cc" "metadata" "--no-deps" "--format-version=1", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "inspect" "9e8c78632efc49d5ca4be88a21d5c6ad019702f3e8f351342d158061c34c78c4", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "exec" "-e" "SOURCE_DIR=/opt/rustwide/workdir" "-e" "CARGO_HOME=/opt/rustwide/cargo-home" "-e" "RUSTUP_HOME=/opt/rustwide/rustup-home" "-e" "CARGO_TARGET_DIR=/opt/rustwide/target" "-e" "CARGO_INCREMENTAL=0" "-e" "RUST_BACKTRACE=full" "-e" "RUSTFLAGS=--cap-lints=forbid" "-e" "RUSTDOCFLAGS=--cap-lints=forbid" "-w" "/opt/rustwide/workdir" "--user" "0:0" "9e8c78632efc49d5ca4be88a21d5c6ad019702f3e8f351342d158061c34c78c4" "/opt/rustwide/cargo-home/bin/cargo" "+a5689b034ad287112f709693384a37f1377eb9cc" "check" "--frozen" "--all" "--all-targets" "--message-format=json", kill_on_drop: false }`
[INFO] [stderr]    Compiling syn v2.0.69
[INFO] [stderr]     Checking anstyle v1.0.11
[INFO] [stderr]     Checking clap_lex v0.7.5
[INFO] [stderr]    Compiling thiserror v1.0.61
[INFO] [stderr]     Checking getrandom v0.3.3
[INFO] [stderr]     Checking anstream v0.6.20
[INFO] [stderr]     Checking uuid v1.17.0
[INFO] [stderr]     Checking clap_builder v4.5.42
[INFO] [stderr]    Compiling clap_derive v4.5.41
[INFO] [stderr]    Compiling thiserror-impl v1.0.61
[INFO] [stderr]    Compiling tokio-macros v2.3.0
[INFO] [stderr]     Checking tokio v1.38.0
[INFO] [stderr]     Checking clap v4.5.42
[INFO] [stderr]     Checking codecrafters-redis v0.1.0 (/opt/rustwide/workdir)
[INFO] [stdout] warning: unused variable: `pos`
[INFO] [stdout]   --> src/rdb/rdb.rs:29:26
[INFO] [stdout]    |
[INFO] [stdout] 29 |         let (key_values, pos) = Self::extract_key_values(&rdb_data, pos + 1);
[INFO] [stdout]    |                          ^^^ help: if this is intentional, prefix it with an underscore: `_pos`
[INFO] [stdout]    |
[INFO] [stdout]    = note: `#[warn(unused_variables)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `val_type`
[INFO] [stdout]    --> src/rdb/rdb.rs:165:17
[INFO] [stdout]     |
[INFO] [stdout] 165 |             let val_type = rdb_data[cur_pos] as usize;
[INFO] [stdout]     |                 ^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_val_type`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: field `pattern` is never read
[INFO] [stdout]  --> src/commands/keys.rs:6:5
[INFO] [stdout]   |
[INFO] [stdout] 5 | pub struct KeysCommand {
[INFO] [stdout]   |            ----------- field in this struct
[INFO] [stdout] 6 |     pattern: String,
[INFO] [stdout]   |     ^^^^^^^
[INFO] [stdout]   |
[INFO] [stdout]   = note: `#[warn(dead_code)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: fields `version` and `metadata` are never read
[INFO] [stdout]  --> src/rdb/rdb.rs:8:5
[INFO] [stdout]   |
[INFO] [stdout] 7 | pub struct Rdb {
[INFO] [stdout]   |            --- fields in this struct
[INFO] [stdout] 8 |     version: String,
[INFO] [stdout]   |     ^^^^^^^
[INFO] [stdout] 9 |     metadata: HashMap<String, String>,
[INFO] [stdout]   |     ^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `std::string::String`
[INFO] [stdout]    --> src/redis/client.rs:162:133
[INFO] [stdout]     |
[INFO] [stdout] 162 | ...tring_resp("dir".into()), create_bulk_string_resp(self.rdb_dir.clone().into())])],
[INFO] [stdout]     |                                                      ^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout]     = note: `#[deny(self_type_conversion)]` on by default
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 162 -                                 "dir" => vec![create_array_resp(vec![create_bulk_string_resp("dir".into()), create_bulk_string_resp(self.rdb_dir.clone().into())])],
[INFO] [stdout] 162 +                                 "dir" => vec![create_array_resp(vec![create_bulk_string_resp("dir".into()), create_bulk_string_resp(self.rdb_dir.clone())])],
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 162 -                                 "dir" => vec![create_array_resp(vec![create_bulk_string_resp("dir".into()), create_bulk_string_resp(self.rdb_dir.clone().into())])],
[INFO] [stdout] 162 +                                 "dir" => vec![create_array_resp(vec![create_bulk_string_resp("dir".into()), create_bulk_string_resp(<std::string::String as Into<_>>::into(self.rdb_dir.clone()))])],
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `std::string::String`
[INFO] [stdout]    --> src/redis/client.rs:163:147
[INFO] [stdout]     |
[INFO] [stdout] 163 | ...esp("dbfilename".into()), create_bulk_string_resp(self.rdb_file.clone().into())])],
[INFO] [stdout]     |                                                      ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 163 -                                 "dbfilename" => vec![create_array_resp(vec![create_bulk_string_resp("dbfilename".into()), create_bulk_string_resp(self.rdb_file.clone().into())])],
[INFO] [stdout] 163 +                                 "dbfilename" => vec![create_array_resp(vec![create_bulk_string_resp("dbfilename".into()), create_bulk_string_resp(self.rdb_file.clone())])],
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 163 -                                 "dbfilename" => vec![create_array_resp(vec![create_bulk_string_resp("dbfilename".into()), create_bulk_string_resp(self.rdb_file.clone().into())])],
[INFO] [stdout] 163 +                                 "dbfilename" => vec![create_array_resp(vec![create_bulk_string_resp("dbfilename".into()), create_bulk_string_resp(<std::string::String as Into<_>>::into(self.rdb_file.clone()))])],
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `std::string::String`
[INFO] [stdout]   --> src/commands/xadd.rs:83:59
[INFO] [stdout]    |
[INFO] [stdout] 83 |                 cache_stream.stream.push(StreamItem { id: entry_id.clone().into(), key_vals: kvs });
[INFO] [stdout]    |                                                           ^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]    |
[INFO] [stdout] 83 -                 cache_stream.stream.push(StreamItem { id: entry_id.clone().into(), key_vals: kvs });
[INFO] [stdout] 83 +                 cache_stream.stream.push(StreamItem { id: entry_id.clone(), key_vals: kvs });
[INFO] [stdout]    |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]    |
[INFO] [stdout] 83 -                 cache_stream.stream.push(StreamItem { id: entry_id.clone().into(), key_vals: kvs });
[INFO] [stdout] 83 +                 cache_stream.stream.push(StreamItem { id: <std::string::String as Into<_>>::into(entry_id.clone()), key_vals: kvs });
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `std::string::String`
[INFO] [stdout]   --> src/commands/publish.rs:26:136
[INFO] [stdout]    |
[INFO] [stdout] 26 | ...ng_resp("message".into()), create_bulk_string_resp(self.channel.clone().into()), create_bulk_string_resp(self.message.clone().int...
[INFO] [stdout]    |                                                       ^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]    |
[INFO] [stdout] 26 -                             stream.write_all(create_array_resp(vec![create_bulk_string_resp("message".into()), create_bulk_string_resp(self.channel.clone().into()), create_bulk_string_resp(self.message.clone().into())]).as_bytes()).unwrap();
[INFO] [stdout] 26 +                             stream.write_all(create_array_resp(vec![create_bulk_string_resp("message".into()), create_bulk_string_resp(self.channel.clone()), create_bulk_string_resp(self.message.clone().into())]).as_bytes()).unwrap();
[INFO] [stdout]    |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]    |
[INFO] [stdout] 26 -                             stream.write_all(create_array_resp(vec![create_bulk_string_resp("message".into()), create_bulk_string_resp(self.channel.clone().into()), create_bulk_string_resp(self.message.clone().into())]).as_bytes()).unwrap();
[INFO] [stdout] 26 +                             stream.write_all(create_array_resp(vec![create_bulk_string_resp("message".into()), create_bulk_string_resp(<std::string::String as Into<_>>::into(self.channel.clone())), create_bulk_string_resp(self.message.clone().into())]).as_bytes()).unwrap();
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `std::string::String`
[INFO] [stdout]   --> src/commands/publish.rs:26:190
[INFO] [stdout]    |
[INFO] [stdout] 26 | ...f.channel.clone().into()), create_bulk_string_resp(self.message.clone().into())]).as_bytes()).unwrap();
[INFO] [stdout]    |                                                       ^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]    |
[INFO] [stdout] 26 -                             stream.write_all(create_array_resp(vec![create_bulk_string_resp("message".into()), create_bulk_string_resp(self.channel.clone().into()), create_bulk_string_resp(self.message.clone().into())]).as_bytes()).unwrap();
[INFO] [stdout] 26 +                             stream.write_all(create_array_resp(vec![create_bulk_string_resp("message".into()), create_bulk_string_resp(self.channel.clone().into()), create_bulk_string_resp(self.message.clone())]).as_bytes()).unwrap();
[INFO] [stdout]    |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]    |
[INFO] [stdout] 26 -                             stream.write_all(create_array_resp(vec![create_bulk_string_resp("message".into()), create_bulk_string_resp(self.channel.clone().into()), create_bulk_string_resp(self.message.clone().into())]).as_bytes()).unwrap();
[INFO] [stdout] 26 +                             stream.write_all(create_array_resp(vec![create_bulk_string_resp("message".into()), create_bulk_string_resp(self.channel.clone().into()), create_bulk_string_resp(<std::string::String as Into<_>>::into(self.message.clone()))]).as_bytes()).unwrap();
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `std::string::String`
[INFO] [stdout]   --> src/commands/unsubscribe.rs:27:115
[INFO] [stdout]    |
[INFO] [stdout] 27 | ...p("unsubscribe".into()), create_bulk_string_resp(self.channel.to_string().into()), create_int_resp(self.num_subscribed_channels)])];
[INFO] [stdout]    |                                                     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]    |
[INFO] [stdout] 27 -         return vec![create_array_resp(vec![create_bulk_string_resp("unsubscribe".into()), create_bulk_string_resp(self.channel.to_string().into()), create_int_resp(self.num_subscribed_channels)])];
[INFO] [stdout] 27 +         return vec![create_array_resp(vec![create_bulk_string_resp("unsubscribe".into()), create_bulk_string_resp(self.channel.to_string()), create_int_resp(self.num_subscribed_channels)])];
[INFO] [stdout]    |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]    |
[INFO] [stdout] 27 -         return vec![create_array_resp(vec![create_bulk_string_resp("unsubscribe".into()), create_bulk_string_resp(self.channel.to_string().into()), create_int_resp(self.num_subscribed_channels)])];
[INFO] [stdout] 27 +         return vec![create_array_resp(vec![create_bulk_string_resp("unsubscribe".into()), create_bulk_string_resp(<std::string::String as Into<_>>::into(self.channel.to_string())), create_int_resp(self.num_subscribed_channels)])];
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `std::string::String`
[INFO] [stdout]   --> src/commands/subscribe.rs:29:113
[INFO] [stdout]    |
[INFO] [stdout] 29 | ...esp("subscribe".into()), create_bulk_string_resp(self.channel.to_string().into()), create_int_resp(self.num_subscribed_channels)])];
[INFO] [stdout]    |                                                     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]    |
[INFO] [stdout] 29 -         return vec![create_array_resp(vec![create_bulk_string_resp("subscribe".into()), create_bulk_string_resp(self.channel.to_string().into()), create_int_resp(self.num_subscribed_channels)])];
[INFO] [stdout] 29 +         return vec![create_array_resp(vec![create_bulk_string_resp("subscribe".into()), create_bulk_string_resp(self.channel.to_string()), create_int_resp(self.num_subscribed_channels)])];
[INFO] [stdout]    |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]    |
[INFO] [stdout] 29 -         return vec![create_array_resp(vec![create_bulk_string_resp("subscribe".into()), create_bulk_string_resp(self.channel.to_string().into()), create_int_resp(self.num_subscribed_channels)])];
[INFO] [stdout] 29 +         return vec![create_array_resp(vec![create_bulk_string_resp("subscribe".into()), create_bulk_string_resp(<std::string::String as Into<_>>::into(self.channel.to_string())), create_int_resp(self.num_subscribed_channels)])];
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stderr] error: could not compile `codecrafters-redis` (bin "codecrafters-redis") due to 7 previous errors; 4 warnings emitted
[INFO] [stderr] warning: build failed, waiting for other jobs to finish...
[INFO] [stdout] warning: unused variable: `pos`
[INFO] [stdout]   --> src/rdb/rdb.rs:29:26
[INFO] [stdout]    |
[INFO] [stdout] 29 |         let (key_values, pos) = Self::extract_key_values(&rdb_data, pos + 1);
[INFO] [stdout]    |                          ^^^ help: if this is intentional, prefix it with an underscore: `_pos`
[INFO] [stdout]    |
[INFO] [stdout]    = note: `#[warn(unused_variables)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `val_type`
[INFO] [stdout]    --> src/rdb/rdb.rs:165:17
[INFO] [stdout]     |
[INFO] [stdout] 165 |             let val_type = rdb_data[cur_pos] as usize;
[INFO] [stdout]     |                 ^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_val_type`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: field `pattern` is never read
[INFO] [stdout]  --> src/commands/keys.rs:6:5
[INFO] [stdout]   |
[INFO] [stdout] 5 | pub struct KeysCommand {
[INFO] [stdout]   |            ----------- field in this struct
[INFO] [stdout] 6 |     pattern: String,
[INFO] [stdout]   |     ^^^^^^^
[INFO] [stdout]   |
[INFO] [stdout]   = note: `#[warn(dead_code)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: field `metadata` is never read
[INFO] [stdout]  --> src/rdb/rdb.rs:9:5
[INFO] [stdout]   |
[INFO] [stdout] 7 | pub struct Rdb {
[INFO] [stdout]   |            --- field in this struct
[INFO] [stdout] 8 |     version: String,
[INFO] [stdout] 9 |     metadata: HashMap<String, String>,
[INFO] [stdout]   |     ^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `std::string::String`
[INFO] [stdout]    --> src/redis/client.rs:162:133
[INFO] [stdout]     |
[INFO] [stdout] 162 | ...tring_resp("dir".into()), create_bulk_string_resp(self.rdb_dir.clone().into())])],
[INFO] [stdout]     |                                                      ^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout]     = note: `#[deny(self_type_conversion)]` on by default
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 162 -                                 "dir" => vec![create_array_resp(vec![create_bulk_string_resp("dir".into()), create_bulk_string_resp(self.rdb_dir.clone().into())])],
[INFO] [stdout] 162 +                                 "dir" => vec![create_array_resp(vec![create_bulk_string_resp("dir".into()), create_bulk_string_resp(self.rdb_dir.clone())])],
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 162 -                                 "dir" => vec![create_array_resp(vec![create_bulk_string_resp("dir".into()), create_bulk_string_resp(self.rdb_dir.clone().into())])],
[INFO] [stdout] 162 +                                 "dir" => vec![create_array_resp(vec![create_bulk_string_resp("dir".into()), create_bulk_string_resp(<std::string::String as Into<_>>::into(self.rdb_dir.clone()))])],
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `std::string::String`
[INFO] [stdout]    --> src/redis/client.rs:163:147
[INFO] [stdout]     |
[INFO] [stdout] 163 | ...esp("dbfilename".into()), create_bulk_string_resp(self.rdb_file.clone().into())])],
[INFO] [stdout]     |                                                      ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 163 -                                 "dbfilename" => vec![create_array_resp(vec![create_bulk_string_resp("dbfilename".into()), create_bulk_string_resp(self.rdb_file.clone().into())])],
[INFO] [stdout] 163 +                                 "dbfilename" => vec![create_array_resp(vec![create_bulk_string_resp("dbfilename".into()), create_bulk_string_resp(self.rdb_file.clone())])],
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 163 -                                 "dbfilename" => vec![create_array_resp(vec![create_bulk_string_resp("dbfilename".into()), create_bulk_string_resp(self.rdb_file.clone().into())])],
[INFO] [stdout] 163 +                                 "dbfilename" => vec![create_array_resp(vec![create_bulk_string_resp("dbfilename".into()), create_bulk_string_resp(<std::string::String as Into<_>>::into(self.rdb_file.clone()))])],
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `&str`
[INFO] [stdout]     --> src/redis/client.rs:1177:31
[INFO] [stdout]      |
[INFO] [stdout] 1177 |         match cache_guard.get("list_key".into()) {
[INFO] [stdout]      |                               ^^^^^^^^^^^^^^^^^
[INFO] [stdout]      |
[INFO] [stdout]      = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]      |
[INFO] [stdout] 1177 -         match cache_guard.get("list_key".into()) {
[INFO] [stdout] 1177 +         match cache_guard.get("list_key") {
[INFO] [stdout]      |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]      |
[INFO] [stdout] 1177 -         match cache_guard.get("list_key".into()) {
[INFO] [stdout] 1177 +         match cache_guard.get(<&str as Into<_>>::into("list_key")) {
[INFO] [stdout]      |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `&str`
[INFO] [stdout]     --> src/redis/client.rs:1199:31
[INFO] [stdout]      |
[INFO] [stdout] 1199 |         match cache_guard.get("list_key".into()) {
[INFO] [stdout]      |                               ^^^^^^^^^^^^^^^^^
[INFO] [stdout]      |
[INFO] [stdout]      = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]      |
[INFO] [stdout] 1199 -         match cache_guard.get("list_key".into()) {
[INFO] [stdout] 1199 +         match cache_guard.get("list_key") {
[INFO] [stdout]      |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]      |
[INFO] [stdout] 1199 -         match cache_guard.get("list_key".into()) {
[INFO] [stdout] 1199 +         match cache_guard.get(<&str as Into<_>>::into("list_key")) {
[INFO] [stdout]      |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `&str`
[INFO] [stdout]     --> src/redis/client.rs:1218:31
[INFO] [stdout]      |
[INFO] [stdout] 1218 |         match cache_guard.get("list_key".into()) {
[INFO] [stdout]      |                               ^^^^^^^^^^^^^^^^^
[INFO] [stdout]      |
[INFO] [stdout]      = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]      |
[INFO] [stdout] 1218 -         match cache_guard.get("list_key".into()) {
[INFO] [stdout] 1218 +         match cache_guard.get("list_key") {
[INFO] [stdout]      |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]      |
[INFO] [stdout] 1218 -         match cache_guard.get("list_key".into()) {
[INFO] [stdout] 1218 +         match cache_guard.get(<&str as Into<_>>::into("list_key")) {
[INFO] [stdout]      |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `&str`
[INFO] [stdout]     --> src/redis/client.rs:1242:31
[INFO] [stdout]      |
[INFO] [stdout] 1242 |         match cache_gaurd.get("list_key".into()) {
[INFO] [stdout]      |                               ^^^^^^^^^^^^^^^^^
[INFO] [stdout]      |
[INFO] [stdout]      = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]      |
[INFO] [stdout] 1242 -         match cache_gaurd.get("list_key".into()) {
[INFO] [stdout] 1242 +         match cache_gaurd.get("list_key") {
[INFO] [stdout]      |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]      |
[INFO] [stdout] 1242 -         match cache_gaurd.get("list_key".into()) {
[INFO] [stdout] 1242 +         match cache_gaurd.get(<&str as Into<_>>::into("list_key")) {
[INFO] [stdout]      |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `&str`
[INFO] [stdout]     --> src/redis/client.rs:1267:31
[INFO] [stdout]      |
[INFO] [stdout] 1267 |         match cache_guard.get("list_key".into()) {
[INFO] [stdout]      |                               ^^^^^^^^^^^^^^^^^
[INFO] [stdout]      |
[INFO] [stdout]      = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]      |
[INFO] [stdout] 1267 -         match cache_guard.get("list_key".into()) {
[INFO] [stdout] 1267 +         match cache_guard.get("list_key") {
[INFO] [stdout]      |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]      |
[INFO] [stdout] 1267 -         match cache_guard.get("list_key".into()) {
[INFO] [stdout] 1267 +         match cache_guard.get(<&str as Into<_>>::into("list_key")) {
[INFO] [stdout]      |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `&str`
[INFO] [stdout]     --> src/redis/client.rs:1286:31
[INFO] [stdout]      |
[INFO] [stdout] 1286 |         match cache_guard.get("list_key".into()) {
[INFO] [stdout]      |                               ^^^^^^^^^^^^^^^^^
[INFO] [stdout]      |
[INFO] [stdout]      = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]      |
[INFO] [stdout] 1286 -         match cache_guard.get("list_key".into()) {
[INFO] [stdout] 1286 +         match cache_guard.get("list_key") {
[INFO] [stdout]      |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]      |
[INFO] [stdout] 1286 -         match cache_guard.get("list_key".into()) {
[INFO] [stdout] 1286 +         match cache_guard.get(<&str as Into<_>>::into("list_key")) {
[INFO] [stdout]      |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `std::string::String`
[INFO] [stdout]   --> src/commands/xadd.rs:83:59
[INFO] [stdout]    |
[INFO] [stdout] 83 |                 cache_stream.stream.push(StreamItem { id: entry_id.clone().into(), key_vals: kvs });
[INFO] [stdout]    |                                                           ^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]    |
[INFO] [stdout] 83 -                 cache_stream.stream.push(StreamItem { id: entry_id.clone().into(), key_vals: kvs });
[INFO] [stdout] 83 +                 cache_stream.stream.push(StreamItem { id: entry_id.clone(), key_vals: kvs });
[INFO] [stdout]    |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]    |
[INFO] [stdout] 83 -                 cache_stream.stream.push(StreamItem { id: entry_id.clone().into(), key_vals: kvs });
[INFO] [stdout] 83 +                 cache_stream.stream.push(StreamItem { id: <std::string::String as Into<_>>::into(entry_id.clone()), key_vals: kvs });
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `std::string::String`
[INFO] [stdout]   --> src/commands/publish.rs:26:136
[INFO] [stdout]    |
[INFO] [stdout] 26 | ...ng_resp("message".into()), create_bulk_string_resp(self.channel.clone().into()), create_bulk_string_resp(self.message.clone().int...
[INFO] [stdout]    |                                                       ^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]    |
[INFO] [stdout] 26 -                             stream.write_all(create_array_resp(vec![create_bulk_string_resp("message".into()), create_bulk_string_resp(self.channel.clone().into()), create_bulk_string_resp(self.message.clone().into())]).as_bytes()).unwrap();
[INFO] [stdout] 26 +                             stream.write_all(create_array_resp(vec![create_bulk_string_resp("message".into()), create_bulk_string_resp(self.channel.clone()), create_bulk_string_resp(self.message.clone().into())]).as_bytes()).unwrap();
[INFO] [stdout]    |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]    |
[INFO] [stdout] 26 -                             stream.write_all(create_array_resp(vec![create_bulk_string_resp("message".into()), create_bulk_string_resp(self.channel.clone().into()), create_bulk_string_resp(self.message.clone().into())]).as_bytes()).unwrap();
[INFO] [stdout] 26 +                             stream.write_all(create_array_resp(vec![create_bulk_string_resp("message".into()), create_bulk_string_resp(<std::string::String as Into<_>>::into(self.channel.clone())), create_bulk_string_resp(self.message.clone().into())]).as_bytes()).unwrap();
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `std::string::String`
[INFO] [stdout]   --> src/commands/publish.rs:26:190
[INFO] [stdout]    |
[INFO] [stdout] 26 | ...f.channel.clone().into()), create_bulk_string_resp(self.message.clone().into())]).as_bytes()).unwrap();
[INFO] [stdout]    |                                                       ^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]    |
[INFO] [stdout] 26 -                             stream.write_all(create_array_resp(vec![create_bulk_string_resp("message".into()), create_bulk_string_resp(self.channel.clone().into()), create_bulk_string_resp(self.message.clone().into())]).as_bytes()).unwrap();
[INFO] [stdout] 26 +                             stream.write_all(create_array_resp(vec![create_bulk_string_resp("message".into()), create_bulk_string_resp(self.channel.clone().into()), create_bulk_string_resp(self.message.clone())]).as_bytes()).unwrap();
[INFO] [stdout]    |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]    |
[INFO] [stdout] 26 -                             stream.write_all(create_array_resp(vec![create_bulk_string_resp("message".into()), create_bulk_string_resp(self.channel.clone().into()), create_bulk_string_resp(self.message.clone().into())]).as_bytes()).unwrap();
[INFO] [stdout] 26 +                             stream.write_all(create_array_resp(vec![create_bulk_string_resp("message".into()), create_bulk_string_resp(self.channel.clone().into()), create_bulk_string_resp(<std::string::String as Into<_>>::into(self.message.clone()))]).as_bytes()).unwrap();
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `std::string::String`
[INFO] [stdout]   --> src/commands/unsubscribe.rs:27:115
[INFO] [stdout]    |
[INFO] [stdout] 27 | ...p("unsubscribe".into()), create_bulk_string_resp(self.channel.to_string().into()), create_int_resp(self.num_subscribed_channels)])];
[INFO] [stdout]    |                                                     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]    |
[INFO] [stdout] 27 -         return vec![create_array_resp(vec![create_bulk_string_resp("unsubscribe".into()), create_bulk_string_resp(self.channel.to_string().into()), create_int_resp(self.num_subscribed_channels)])];
[INFO] [stdout] 27 +         return vec![create_array_resp(vec![create_bulk_string_resp("unsubscribe".into()), create_bulk_string_resp(self.channel.to_string()), create_int_resp(self.num_subscribed_channels)])];
[INFO] [stdout]    |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]    |
[INFO] [stdout] 27 -         return vec![create_array_resp(vec![create_bulk_string_resp("unsubscribe".into()), create_bulk_string_resp(self.channel.to_string().into()), create_int_resp(self.num_subscribed_channels)])];
[INFO] [stdout] 27 +         return vec![create_array_resp(vec![create_bulk_string_resp("unsubscribe".into()), create_bulk_string_resp(<std::string::String as Into<_>>::into(self.channel.to_string())), create_int_resp(self.num_subscribed_channels)])];
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `std::string::String`
[INFO] [stdout]   --> src/commands/subscribe.rs:29:113
[INFO] [stdout]    |
[INFO] [stdout] 29 | ...esp("subscribe".into()), create_bulk_string_resp(self.channel.to_string().into()), create_int_resp(self.num_subscribed_channels)])];
[INFO] [stdout]    |                                                     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]    |
[INFO] [stdout] 29 -         return vec![create_array_resp(vec![create_bulk_string_resp("subscribe".into()), create_bulk_string_resp(self.channel.to_string().into()), create_int_resp(self.num_subscribed_channels)])];
[INFO] [stdout] 29 +         return vec![create_array_resp(vec![create_bulk_string_resp("subscribe".into()), create_bulk_string_resp(self.channel.to_string()), create_int_resp(self.num_subscribed_channels)])];
[INFO] [stdout]    |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]    |
[INFO] [stdout] 29 -         return vec![create_array_resp(vec![create_bulk_string_resp("subscribe".into()), create_bulk_string_resp(self.channel.to_string().into()), create_int_resp(self.num_subscribed_channels)])];
[INFO] [stdout] 29 +         return vec![create_array_resp(vec![create_bulk_string_resp("subscribe".into()), create_bulk_string_resp(<std::string::String as Into<_>>::into(self.channel.to_string())), create_int_resp(self.num_subscribed_channels)])];
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stderr] error: could not compile `codecrafters-redis` (bin "codecrafters-redis" test) due to 13 previous errors; 4 warnings emitted
[INFO] [stderr] warning: unused dependency `anyhow`
[INFO] [stderr]  --> Cargo.toml:2:1
[INFO] [stderr]   |
[INFO] [stderr] 2 | anyhow = "1.0.59"
[INFO] [stderr]   | ^^^^^^^^^^^^^^^^^
[INFO] [stderr]   |
[INFO] [stderr]   = note: `cargo::unused_dependencies` is set to `warn` by default
[INFO] [stderr] help: consider removing the dependency on `anyhow`
[INFO] [stderr] warning: unused dependency `thiserror`
[INFO] [stderr]  --> Cargo.toml:4:1
[INFO] [stderr]   |
[INFO] [stderr] 4 | thiserror = "1.0.32"
[INFO] [stderr]   | ^^^^^^^^^^^^^^^^^^^^
[INFO] [stderr]   |
[INFO] [stderr] help: consider removing the dependency on `thiserror`
[INFO] [stderr] warning: unused dependency `tokio`
[INFO] [stderr]   --> Cargo.toml:10:15
[INFO] [stderr]    |
[INFO] [stderr] 10 | [dependencies.tokio]
[INFO] [stderr]    |               ^^^^^^
[INFO] [stderr]    |
[INFO] [stderr] help: consider removing the dependency on `tokio`
[INFO] [stderr] warning: `codecrafters-redis` (manifest) generated 3 warnings
[INFO] running `Command { std: "docker" "inspect" "9e8c78632efc49d5ca4be88a21d5c6ad019702f3e8f351342d158061c34c78c4", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "rm" "-f" "9e8c78632efc49d5ca4be88a21d5c6ad019702f3e8f351342d158061c34c78c4", kill_on_drop: false }`
[INFO] [stdout] 9e8c78632efc49d5ca4be88a21d5c6ad019702f3e8f351342d158061c34c78c4
