Treat SKIPPED as a final task status.

Previously we were showing them as failed.

Author: macdice

cfbot_web.py

@@ -439,7 +439,7 @@ def build_page(
                         html = svg_img("new_failure")
                     else:
                         html = svg_img("old_failure")
-                elif build_result.status in ("PAUSED"):
+                elif build_result.status in ("PAUSED", "SKIPPED"):
                     html = svg_img("paused")
                 elif build_result.status in ("CREATED", "SCHEDULED", "PAUSED"):
                     html = svg_img("waiting_to_start")

create.sql

@@ -37,7 +37,7 @@ ALTER FUNCTION public.build_status_running(status text) OWNER TO cfbot;
 CREATE FUNCTION public.task_status_running(status text) RETURNS boolean
     LANGUAGE sql IMMUTABLE STRICT PARALLEL SAFE
     BEGIN ATOMIC
- SELECT (status = ANY (ARRAY['CREATED'::text, 'TRIGGERED'::text, 'SCHEDULED'::text, 'EXECUTING'::text]));
+ SELECT (status = ANY (ARRAY['CREATED'::text, 'TRIGGERED'::text, 'SCHEDULED'::text, 'EXECUTING'::text, 'SKIPPED'::text]));
 END;