From b37c45ad2a295f067b6cacf9424ff9f68674f82b Mon Sep 17 00:00:00 2001
From: Jorge Aparicio <jorge@japaric.io>
Date: Thu, 27 Jul 2017 12:11:22 -0500
Subject: [PATCH] make task.$T.priority optional

default the value to 1 if omitted
---
 macros/src/check.rs | 28 ++++++++--------------------
 1 file changed, 8 insertions(+), 20 deletions(-)

diff --git a/macros/src/check.rs b/macros/src/check.rs
index cedf933..571f235 100644
--- a/macros/src/check.rs
+++ b/macros/src/check.rs
@@ -31,16 +31,8 @@ pub fn app(app: check::App) -> Result<App> {
         resources: app.resources,
         tasks: app.tasks
             .into_iter()
-            .map(|(k, v)| {
-                let name = k.clone();
-                Ok((
-                    k,
-                    ::check::task(v)
-                        .chain_err(|| format!("checking task `{}`", name))?,
-                ))
-            })
-            .collect::<Result<_>>()
-            .chain_err(|| "checking `tasks`")?,
+            .map(|(k, v)| (k, ::check::task(v)))
+            .collect(),
     };
 
     ::check::resources(&app)
@@ -68,15 +60,11 @@ fn resources(app: &App) -> Result<()> {
     Ok(())
 }
 
-fn task(task: syntax::check::Task) -> Result<Task> {
-    if let Some(priority) = task.priority {
-        Ok(Task {
-            enabled: task.enabled,
-            path: task.path,
-            priority,
-            resources: task.resources,
-        })
-    } else {
-        bail!("should contain a `priority` field")
+fn task(task: syntax::check::Task) -> Task {
+    Task {
+        enabled: task.enabled,
+        path: task.path,
+        priority: task.priority.unwrap_or(1),
+        resources: task.resources,
     }
 }
-- 
GitLab