[INFO] updating cached repository itkubsa1/bprocess [INFO] running `"git" "fetch" "--all"` [INFO] [stdout] Fetching origin [INFO] [stderr] From git://github.com/itkubsa1/bprocess [INFO] [stderr] * branch HEAD -> FETCH_HEAD [INFO] running `"git" "clone" "work/cache/sources/gh/itkubsa1/bprocess" "work/ex/clippy-test-run/sources/stable/gh/itkubsa1/bprocess"` [INFO] [stderr] Cloning into 'work/ex/clippy-test-run/sources/stable/gh/itkubsa1/bprocess'... [INFO] [stderr] done. [INFO] running `"git" "clone" "work/cache/sources/gh/itkubsa1/bprocess" "work/ex/clippy-test-run/sources/stable+rustflags=-Dclippy%3A%3Ainto_iter_on_array/gh/itkubsa1/bprocess"` [INFO] [stderr] Cloning into 'work/ex/clippy-test-run/sources/stable+rustflags=-Dclippy%3A%3Ainto_iter_on_array/gh/itkubsa1/bprocess'... [INFO] [stderr] done. [INFO] running `"git" "rev-parse" "HEAD"` [INFO] [stdout] c1eccca0dca08a2804be13e2ecdfa0e5a3dcb48d [INFO] sha for GitHub repo itkubsa1/bprocess: c1eccca0dca08a2804be13e2ecdfa0e5a3dcb48d [INFO] validating manifest of itkubsa1/bprocess on toolchain stable [INFO] running `"/mnt/big/crater/work/local/cargo-home/bin/cargo" "+stable" "read-manifest" "--manifest-path" "Cargo.toml"` [INFO] validating manifest of itkubsa1/bprocess on toolchain stable+rustflags=-Dclippy::into_iter_on_array [INFO] running `"/mnt/big/crater/work/local/cargo-home/bin/cargo" "+stable" "read-manifest" "--manifest-path" "Cargo.toml"` [INFO] started frobbing itkubsa1/bprocess [INFO] finished frobbing itkubsa1/bprocess [INFO] frobbed toml for itkubsa1/bprocess written to work/ex/clippy-test-run/sources/stable/gh/itkubsa1/bprocess/Cargo.toml [INFO] started frobbing itkubsa1/bprocess [INFO] finished frobbing itkubsa1/bprocess [INFO] frobbed toml for itkubsa1/bprocess written to work/ex/clippy-test-run/sources/stable+rustflags=-Dclippy%3A%3Ainto_iter_on_array/gh/itkubsa1/bprocess/Cargo.toml [INFO] crate itkubsa1/bprocess has a lockfile. skipping [INFO] running `"/mnt/big/crater/work/local/cargo-home/bin/cargo" "+stable" "fetch" "--locked" "--manifest-path" "Cargo.toml"` [INFO] running `"/mnt/big/crater/work/local/cargo-home/bin/cargo" "+stable" "fetch" "--locked" "--manifest-path" "Cargo.toml"` [INFO] linting itkubsa1/bprocess against stable for clippy-test-run [INFO] running `"docker" "create" "-v" "/mnt/big/crater/work/local/target-dirs/clippy-test-run/worker-7/stable:/opt/crater/target:rw,Z" "-v" "/mnt/big/crater/work/ex/clippy-test-run/sources/stable/gh/itkubsa1/bprocess:/opt/crater/workdir:ro,Z" "-v" "/mnt/big/crater/work/local/cargo-home:/opt/crater/cargo-home:ro,Z" "-v" "/mnt/big/crater/work/local/rustup-home:/opt/crater/rustup-home:ro,Z" "-e" "USER_ID=1000" "-e" "SOURCE_DIR=/opt/crater/workdir" "-e" "MAP_USER_ID=1000" "-e" "CARGO_TARGET_DIR=/opt/crater/target" "-e" "CARGO_INCREMENTAL=0" "-e" "RUST_BACKTRACE=full" "-e" "RUSTFLAGS=--cap-lints=forbid" "-e" "CARGO_HOME=/opt/crater/cargo-home" "-e" "RUSTUP_HOME=/opt/crater/rustup-home" "-w" "/opt/crater/workdir" "-m" "1536M" "--network" "none" "rustops/crates-build-env" "/opt/crater/cargo-home/bin/cargo" "+stable" "clippy" "--frozen" "--all" "--all-targets"` [INFO] [stdout] 6a777077ac195712b39602da898cb7d1007638974faf27d62617bcb9c515f68c [INFO] running `"docker" "start" "-a" "6a777077ac195712b39602da898cb7d1007638974faf27d62617bcb9c515f68c"` [INFO] [stderr] Checking fs_extra v1.1.0 [INFO] [stderr] Checking Rust-firststeps v0.1.0 (/opt/crater/workdir) [INFO] [stderr] warning: unneeded return statement [INFO] [stderr] --> src/main.rs:75:18 [INFO] [stderr] | [INFO] [stderr] 75 | Ok(n) => return (n.as_secs() / SECS_IN_DAY) as u64, [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: remove `return` as shown: `(n.as_secs() / SECS_IN_DAY) as u64` [INFO] [stderr] | [INFO] [stderr] = note: #[warn(clippy::needless_return)] on by default [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return [INFO] [stderr] [INFO] [stderr] warning: unneeded return statement [INFO] [stderr] --> src/main.rs:128:5 [INFO] [stderr] | [INFO] [stderr] 128 | return result; [INFO] [stderr] | ^^^^^^^^^^^^^^ help: remove `return` as shown: `result` [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return [INFO] [stderr] [INFO] [stderr] warning: unneeded return statement [INFO] [stderr] --> src/main.rs:75:18 [INFO] [stderr] | [INFO] [stderr] 75 | Ok(n) => return (n.as_secs() / SECS_IN_DAY) as u64, [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: remove `return` as shown: `(n.as_secs() / SECS_IN_DAY) as u64` [INFO] [stderr] | [INFO] [stderr] = note: #[warn(clippy::needless_return)] on by default [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return [INFO] [stderr] [INFO] [stderr] warning: unneeded return statement [INFO] [stderr] --> src/main.rs:128:5 [INFO] [stderr] | [INFO] [stderr] 128 | return result; [INFO] [stderr] | ^^^^^^^^^^^^^^ help: remove `return` as shown: `result` [INFO] [stderr] | [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_return [INFO] [stderr] [INFO] [stderr] warning: unused variable: `parsed` [INFO] [stderr] --> src/main.rs:132:9 [INFO] [stderr] | [INFO] [stderr] 132 | let parsed = match arg.parse::() { [INFO] [stderr] | ^^^^^^ help: consider using `_parsed` instead [INFO] [stderr] | [INFO] [stderr] = note: #[warn(unused_variables)] on by default [INFO] [stderr] [INFO] [stderr] warning: unused variable: `n` [INFO] [stderr] --> src/main.rs:133:12 [INFO] [stderr] | [INFO] [stderr] 133 | Ok(n) => return Ok(()), [INFO] [stderr] | ^ help: consider using `_n` instead [INFO] [stderr] [INFO] [stderr] warning: variable `wfilepath` is assigned to, but never used [INFO] [stderr] --> src/main.rs:340:13 [INFO] [stderr] | [INFO] [stderr] 340 | let mut wfilepath: String = "".to_string(); [INFO] [stderr] | ^^^^^^^^^ [INFO] [stderr] | [INFO] [stderr] = note: consider using `_wfilepath` instead [INFO] [stderr] [INFO] [stderr] warning: value assigned to `wfilepath` is never read [INFO] [stderr] --> src/main.rs:347:9 [INFO] [stderr] | [INFO] [stderr] 347 | wfilepath = value.clone().to_string(); [INFO] [stderr] | ^^^^^^^^^ [INFO] [stderr] | [INFO] [stderr] = note: #[warn(unused_assignments)] on by default [INFO] [stderr] [INFO] [stderr] warning: crate `Rust_firststeps` should have a snake case name such as `rust_firststeps` [INFO] [stderr] | [INFO] [stderr] = note: #[warn(non_snake_case)] on by default [INFO] [stderr] [INFO] [stderr] warning: Err(_) will match all errors, maybe not a good idea [INFO] [stderr] --> src/main.rs:76:9 [INFO] [stderr] | [INFO] [stderr] 76 | Err(_) => panic!("Err: Cant calculate age of file. Aborting!"), [INFO] [stderr] | ^^^^^^ [INFO] [stderr] | [INFO] [stderr] = note: #[warn(clippy::match_wild_err_arm)] on by default [INFO] [stderr] = note: to remove this warning, match each error separately or use unreachable macro [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#match_wild_err_arm [INFO] [stderr] [INFO] [stderr] warning: this `.filter_map` can be written more simply using `.filter` [INFO] [stderr] --> src/main.rs:92:5 [INFO] [stderr] | [INFO] [stderr] 92 | / path.filter_map(Result::ok) [INFO] [stderr] 93 | | .filter_map(|e| { [INFO] [stderr] 94 | | if e.path() [INFO] [stderr] 95 | | .metadata() [INFO] [stderr] ... | [INFO] [stderr] 103 | | } [INFO] [stderr] 104 | | }) [INFO] [stderr] | |__________^ [INFO] [stderr] | [INFO] [stderr] = note: #[warn(clippy::unnecessary_filter_map)] on by default [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#unnecessary_filter_map [INFO] [stderr] [INFO] [stderr] warning: this let-binding has unit value. Consider omitting `let parsed =` [INFO] [stderr] --> src/main.rs:132:5 [INFO] [stderr] | [INFO] [stderr] 132 | / let parsed = match arg.parse::() { [INFO] [stderr] 133 | | Ok(n) => return Ok(()), [INFO] [stderr] 134 | | Err(_) => return Err(String::from("Invalid value for days in month")), [INFO] [stderr] 135 | | }; [INFO] [stderr] | |______^ [INFO] [stderr] | [INFO] [stderr] = note: #[warn(clippy::let_unit_value)] on by default [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#let_unit_value [INFO] [stderr] [INFO] [stderr] error: for loop over `iterate_dir("", "_month", "_week").iter().min()`, which is an `Option`. This is more readably written as an `if let` statement. [INFO] [stderr] --> src/main.rs:252:25 [INFO] [stderr] | [INFO] [stderr] 252 | for (key, value) in iterate_dir("", "_month", "_week").iter().min() { [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stderr] | [INFO] [stderr] = note: #[deny(clippy::for_loop_over_option)] on by default [INFO] [stderr] = help: consider replacing `for (key, value) in iterate_dir("", "_month", "_week").iter().min()` with `if let Some((key, value)) = iterate_dir("", "_month", "_week").iter().min()` [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#for_loop_over_option [INFO] [stderr] [INFO] [stderr] error: for loop over `iterate_dir("_month", "_week", "$dummy$_$that$_$never$_$can$_$be$_$mEt$") [INFO] [stderr] .iter() [INFO] [stderr] .min()`, which is an `Option`. This is more readably written as an `if let` statement. [INFO] [stderr] --> src/main.rs:272:25 [INFO] [stderr] | [INFO] [stderr] 272 | for (key, value) in iterate_dir("_month", "_week", "$dummy$_$that$_$never$_$can$_$be$_$mEt$") [INFO] [stderr] | _________________________^ [INFO] [stderr] 273 | | .iter() [INFO] [stderr] 274 | | .min() [INFO] [stderr] | |______________^ [INFO] [stderr] | [INFO] [stderr] = help: consider replacing `for (key, value) in iterate_dir("_month", "_week", "$dummy$_$that$_$never$_$can$_$be$_$mEt$") [INFO] [stderr] .iter() [INFO] [stderr] .min()` with `if let Some((key, value)) = iterate_dir("_month", "_week", "$dummy$_$that$_$never$_$can$_$be$_$mEt$") [INFO] [stderr] .iter() [INFO] [stderr] .min()` [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#for_loop_over_option [INFO] [stderr] [INFO] [stderr] error: for loop over `iterate_dir("_week", "_month", "$dummy$_$that$_$never$_$can$_$be$_$mEt$") [INFO] [stderr] .iter() [INFO] [stderr] .min()`, which is an `Option`. This is more readably written as an `if let` statement. [INFO] [stderr] --> src/main.rs:287:25 [INFO] [stderr] | [INFO] [stderr] 287 | for (key, value) in iterate_dir("_week", "_month", "$dummy$_$that$_$never$_$can$_$be$_$mEt$") [INFO] [stderr] | _________________________^ [INFO] [stderr] 288 | | .iter() [INFO] [stderr] 289 | | .min() [INFO] [stderr] | |______________^ [INFO] [stderr] | [INFO] [stderr] = help: consider replacing `for (key, value) in iterate_dir("_week", "_month", "$dummy$_$that$_$never$_$can$_$be$_$mEt$") [INFO] [stderr] .iter() [INFO] [stderr] .min()` with `if let Some((key, value)) = iterate_dir("_week", "_month", "$dummy$_$that$_$never$_$can$_$be$_$mEt$") [INFO] [stderr] .iter() [INFO] [stderr] .min()` [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#for_loop_over_option [INFO] [stderr] [INFO] [stderr] error: for loop over `iterate_dir("", "_month", "_week").iter().min()`, which is an `Option`. This is more readably written as an `if let` statement. [INFO] [stderr] --> src/main.rs:329:25 [INFO] [stderr] | [INFO] [stderr] 329 | for (key, value) in iterate_dir("", "_month", "_week").iter().min() { [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stderr] | [INFO] [stderr] = help: consider replacing `for (key, value) in iterate_dir("", "_month", "_week").iter().min()` with `if let Some((key, value)) = iterate_dir("", "_month", "_week").iter().min()` [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#for_loop_over_option [INFO] [stderr] [INFO] [stderr] error: for loop over `iterate_dir("_week", "_month", "$dummy$_$that$_$never$_$can$_$be$_$mEt$") [INFO] [stderr] .iter() [INFO] [stderr] .min()`, which is an `Option`. This is more readably written as an `if let` statement. [INFO] [stderr] --> src/main.rs:342:25 [INFO] [stderr] | [INFO] [stderr] 342 | for (key, value) in iterate_dir("_week", "_month", "$dummy$_$that$_$never$_$can$_$be$_$mEt$") [INFO] [stderr] | _________________________^ [INFO] [stderr] 343 | | .iter() [INFO] [stderr] 344 | | .min() [INFO] [stderr] | |______________^ [INFO] [stderr] | [INFO] [stderr] = help: consider replacing `for (key, value) in iterate_dir("_week", "_month", "$dummy$_$that$_$never$_$can$_$be$_$mEt$") [INFO] [stderr] .iter() [INFO] [stderr] .min()` with `if let Some((key, value)) = iterate_dir("_week", "_month", "$dummy$_$that$_$never$_$can$_$be$_$mEt$") [INFO] [stderr] .iter() [INFO] [stderr] .min()` [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#for_loop_over_option [INFO] [stderr] [INFO] [stderr] error: aborting due to 5 previous errors [INFO] [stderr] [INFO] [stderr] error: Could not compile `Rust-firststeps`. [INFO] [stderr] warning: build failed, waiting for other jobs to finish... [INFO] [stderr] warning: unused variable: `parsed` [INFO] [stderr] --> src/main.rs:132:9 [INFO] [stderr] | [INFO] [stderr] 132 | let parsed = match arg.parse::() { [INFO] [stderr] | ^^^^^^ help: consider using `_parsed` instead [INFO] [stderr] | [INFO] [stderr] = note: #[warn(unused_variables)] on by default [INFO] [stderr] [INFO] [stderr] warning: unused variable: `n` [INFO] [stderr] --> src/main.rs:133:12 [INFO] [stderr] | [INFO] [stderr] 133 | Ok(n) => return Ok(()), [INFO] [stderr] | ^ help: consider using `_n` instead [INFO] [stderr] [INFO] [stderr] warning: variable `wfilepath` is assigned to, but never used [INFO] [stderr] --> src/main.rs:340:13 [INFO] [stderr] | [INFO] [stderr] 340 | let mut wfilepath: String = "".to_string(); [INFO] [stderr] | ^^^^^^^^^ [INFO] [stderr] | [INFO] [stderr] = note: consider using `_wfilepath` instead [INFO] [stderr] [INFO] [stderr] warning: value assigned to `wfilepath` is never read [INFO] [stderr] --> src/main.rs:347:9 [INFO] [stderr] | [INFO] [stderr] 347 | wfilepath = value.clone().to_string(); [INFO] [stderr] | ^^^^^^^^^ [INFO] [stderr] | [INFO] [stderr] = note: #[warn(unused_assignments)] on by default [INFO] [stderr] [INFO] [stderr] warning: crate `Rust_firststeps` should have a snake case name such as `rust_firststeps` [INFO] [stderr] | [INFO] [stderr] = note: #[warn(non_snake_case)] on by default [INFO] [stderr] [INFO] [stderr] warning: Err(_) will match all errors, maybe not a good idea [INFO] [stderr] --> src/main.rs:76:9 [INFO] [stderr] | [INFO] [stderr] 76 | Err(_) => panic!("Err: Cant calculate age of file. Aborting!"), [INFO] [stderr] | ^^^^^^ [INFO] [stderr] | [INFO] [stderr] = note: #[warn(clippy::match_wild_err_arm)] on by default [INFO] [stderr] = note: to remove this warning, match each error separately or use unreachable macro [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#match_wild_err_arm [INFO] [stderr] [INFO] [stderr] warning: this `.filter_map` can be written more simply using `.filter` [INFO] [stderr] --> src/main.rs:92:5 [INFO] [stderr] | [INFO] [stderr] 92 | / path.filter_map(Result::ok) [INFO] [stderr] 93 | | .filter_map(|e| { [INFO] [stderr] 94 | | if e.path() [INFO] [stderr] 95 | | .metadata() [INFO] [stderr] ... | [INFO] [stderr] 103 | | } [INFO] [stderr] 104 | | }) [INFO] [stderr] | |__________^ [INFO] [stderr] | [INFO] [stderr] = note: #[warn(clippy::unnecessary_filter_map)] on by default [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#unnecessary_filter_map [INFO] [stderr] [INFO] [stderr] warning: this let-binding has unit value. Consider omitting `let parsed =` [INFO] [stderr] --> src/main.rs:132:5 [INFO] [stderr] | [INFO] [stderr] 132 | / let parsed = match arg.parse::() { [INFO] [stderr] 133 | | Ok(n) => return Ok(()), [INFO] [stderr] 134 | | Err(_) => return Err(String::from("Invalid value for days in month")), [INFO] [stderr] 135 | | }; [INFO] [stderr] | |______^ [INFO] [stderr] | [INFO] [stderr] = note: #[warn(clippy::let_unit_value)] on by default [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#let_unit_value [INFO] [stderr] [INFO] [stderr] error: for loop over `iterate_dir("", "_month", "_week").iter().min()`, which is an `Option`. This is more readably written as an `if let` statement. [INFO] [stderr] --> src/main.rs:252:25 [INFO] [stderr] | [INFO] [stderr] 252 | for (key, value) in iterate_dir("", "_month", "_week").iter().min() { [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stderr] | [INFO] [stderr] = note: #[deny(clippy::for_loop_over_option)] on by default [INFO] [stderr] = help: consider replacing `for (key, value) in iterate_dir("", "_month", "_week").iter().min()` with `if let Some((key, value)) = iterate_dir("", "_month", "_week").iter().min()` [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#for_loop_over_option [INFO] [stderr] [INFO] [stderr] error: for loop over `iterate_dir("_month", "_week", "$dummy$_$that$_$never$_$can$_$be$_$mEt$") [INFO] [stderr] .iter() [INFO] [stderr] .min()`, which is an `Option`. This is more readably written as an `if let` statement. [INFO] [stderr] --> src/main.rs:272:25 [INFO] [stderr] | [INFO] [stderr] 272 | for (key, value) in iterate_dir("_month", "_week", "$dummy$_$that$_$never$_$can$_$be$_$mEt$") [INFO] [stderr] | _________________________^ [INFO] [stderr] 273 | | .iter() [INFO] [stderr] 274 | | .min() [INFO] [stderr] | |______________^ [INFO] [stderr] | [INFO] [stderr] = help: consider replacing `for (key, value) in iterate_dir("_month", "_week", "$dummy$_$that$_$never$_$can$_$be$_$mEt$") [INFO] [stderr] .iter() [INFO] [stderr] .min()` with `if let Some((key, value)) = iterate_dir("_month", "_week", "$dummy$_$that$_$never$_$can$_$be$_$mEt$") [INFO] [stderr] .iter() [INFO] [stderr] .min()` [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#for_loop_over_option [INFO] [stderr] [INFO] [stderr] error: for loop over `iterate_dir("_week", "_month", "$dummy$_$that$_$never$_$can$_$be$_$mEt$") [INFO] [stderr] .iter() [INFO] [stderr] .min()`, which is an `Option`. This is more readably written as an `if let` statement. [INFO] [stderr] --> src/main.rs:287:25 [INFO] [stderr] | [INFO] [stderr] 287 | for (key, value) in iterate_dir("_week", "_month", "$dummy$_$that$_$never$_$can$_$be$_$mEt$") [INFO] [stderr] | _________________________^ [INFO] [stderr] 288 | | .iter() [INFO] [stderr] 289 | | .min() [INFO] [stderr] | |______________^ [INFO] [stderr] | [INFO] [stderr] = help: consider replacing `for (key, value) in iterate_dir("_week", "_month", "$dummy$_$that$_$never$_$can$_$be$_$mEt$") [INFO] [stderr] .iter() [INFO] [stderr] .min()` with `if let Some((key, value)) = iterate_dir("_week", "_month", "$dummy$_$that$_$never$_$can$_$be$_$mEt$") [INFO] [stderr] .iter() [INFO] [stderr] .min()` [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#for_loop_over_option [INFO] [stderr] [INFO] [stderr] error: for loop over `iterate_dir("", "_month", "_week").iter().min()`, which is an `Option`. This is more readably written as an `if let` statement. [INFO] [stderr] --> src/main.rs:329:25 [INFO] [stderr] | [INFO] [stderr] 329 | for (key, value) in iterate_dir("", "_month", "_week").iter().min() { [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stderr] | [INFO] [stderr] = help: consider replacing `for (key, value) in iterate_dir("", "_month", "_week").iter().min()` with `if let Some((key, value)) = iterate_dir("", "_month", "_week").iter().min()` [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#for_loop_over_option [INFO] [stderr] [INFO] [stderr] error: for loop over `iterate_dir("_week", "_month", "$dummy$_$that$_$never$_$can$_$be$_$mEt$") [INFO] [stderr] .iter() [INFO] [stderr] .min()`, which is an `Option`. This is more readably written as an `if let` statement. [INFO] [stderr] --> src/main.rs:342:25 [INFO] [stderr] | [INFO] [stderr] 342 | for (key, value) in iterate_dir("_week", "_month", "$dummy$_$that$_$never$_$can$_$be$_$mEt$") [INFO] [stderr] | _________________________^ [INFO] [stderr] 343 | | .iter() [INFO] [stderr] 344 | | .min() [INFO] [stderr] | |______________^ [INFO] [stderr] | [INFO] [stderr] = help: consider replacing `for (key, value) in iterate_dir("_week", "_month", "$dummy$_$that$_$never$_$can$_$be$_$mEt$") [INFO] [stderr] .iter() [INFO] [stderr] .min()` with `if let Some((key, value)) = iterate_dir("_week", "_month", "$dummy$_$that$_$never$_$can$_$be$_$mEt$") [INFO] [stderr] .iter() [INFO] [stderr] .min()` [INFO] [stderr] = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#for_loop_over_option [INFO] [stderr] [INFO] [stderr] error: aborting due to 5 previous errors [INFO] [stderr] [INFO] [stderr] error: Could not compile `Rust-firststeps`. [INFO] [stderr] [INFO] [stderr] To learn more, run the command again with --verbose. [INFO] running `"docker" "inspect" "6a777077ac195712b39602da898cb7d1007638974faf27d62617bcb9c515f68c"` [INFO] running `"docker" "rm" "-f" "6a777077ac195712b39602da898cb7d1007638974faf27d62617bcb9c515f68c"` [INFO] [stdout] 6a777077ac195712b39602da898cb7d1007638974faf27d62617bcb9c515f68c