Fix clippy: `.map().collect()` can be replaced with `.try_for_each()`

Signed-off-by: Matthias Beyer <mail@beyermatthias.de>

1 files changed, 1 insertions, 2 deletions

diff --git a/src/job/runnable.rs b/src/job/runnable.rs
index 5c10d06..5f8be7a 100644
--- a/src/job/runnable.rs
+++ b/src/job/runnable.rs
@@ -90,14 +90,13 @@ impl RunnableJob {
             let _ = Self::env_resources(job.resources(), job.package().environment().as_ref())
                 .into_iter()
                 .inspect(|(name, _)| debug!("Checking: {}", name))
-                .map(|(name, _)| {
+                .try_for_each(|(name, _)| {
                     if !config.containers().allowed_env().contains(&name) {
                         Err(anyhow!("Environment variable name not allowed: {}", name))
                     } else {
                         Ok(())
                     }
                 })
-                .collect::<Result<()>>()
                 .with_context(|| anyhow!("Checking allowed variables for package {} {}", job.package().name(), job.package().version()))
                 .context("Checking allowed variable names")?;
         } else {