jetbrains formatting

Author: firecow

src/index.ts

@@ -5,6 +5,21 @@ import {Parser} from "./parser";
 
 const colorizer = winston.format.colorize();
 
+// Array polyfill
+declare global {
+    // tslint:disable-next-line:interface-name
+    interface Array<T> {
+        last(): T | undefined;
+        first(): T | undefined;
+    }
+}
+Array.prototype.last = function() {
+    return this[this.length - 1];
+};
+Array.prototype.first = function() {
+    return this[0];
+};
+
 const logger: winston.Logger = winston.createLogger({
     format: winston.format.combine(
         winston.format.timestamp({format: "HH:mm:SS"}),
@@ -44,7 +59,9 @@ const runJobs = async () => {
             await Promise.all(promises);
             console.log("");
         } catch (e) {
-            if (e !== "") { console.error(e); }
+            if (e !== "") {
+                console.error(e);
+            }
             process.exit(1);
         }
     }

src/job.ts

@@ -12,7 +12,7 @@ export class Job {
     private readonly cwd: any;
     private readonly globals: any;
 
-    private readonly variables: {[key: string]: string};
+    private readonly variables: { [key: string]: string };
 
     private readonly allowFailure: boolean;
 
@@ -100,7 +100,7 @@ export class Job {
         return `${c.blueBright(`${this.name}`)} ${mistakeStr}`;
     }
 
-    private getEnvs(): {[key: string]: string} {
+    private getEnvs(): { [key: string]: string } {
         return {...this.globals.variables || {}, ...this.variables, ...process.env};
     }
 
@@ -122,7 +122,10 @@ export class Job {
                 child.stdout.on("data", (buf) => {
                     const lines = `${buf}`.split(/\r?\n/);
                     lines.forEach((l) => {
-                        if (l) { process.stdout.write(`${c.blueBright(`${this.name}`)} ${c.greenBright(`>`)} ${c.green(`${l}`)}\n`); }
+                        if (!l) {
+                            return;
+                        }
+                        process.stdout.write(`${c.blueBright(`${this.name}`)} ${c.greenBright(`>`)} ${c.green(`${l}`)}\n`);
                     });
                 });
             }
@@ -131,7 +134,10 @@ export class Job {
                 child.stderr.on("data", (buf) => {
                     const lines = `${buf}`.split(/\r?\n/);
                     lines.forEach((l) => {
-                        if (l) { process.stderr.write(`${c.blueBright(`${this.name}`)} ${c.redBright(`>`)} ${c.red(`${l}`)}\n`); }
+                        if (!l) {
+                            return;
+                        }
+                        process.stderr.write(`${c.blueBright(`${this.name}`)} ${c.redBright(`>`)} ${c.red(`${l}`)}\n`);
                     });
                 });
             }

src/parser.ts

@@ -1,5 +1,5 @@
 import c = require("ansi-colors");
-import deepExtend  = require("deep-extend");
+import deepExtend = require("deep-extend");
 import fs = require("fs");
 import yaml = require("js-yaml");
 import * as winston from "winston";
@@ -36,11 +36,15 @@ export class Parser {
 
         // Parse .gitlab-ci.yml
         orderedYml.push(yaml.safeLoad(fs.readFileSync(gitlabCiYmlPath, "utf8")));
-        orderedVariables.push(orderedYml[orderedYml.length - 1].variables || {});
+        if (!orderedYml.last()) { // Print if empty
+            console.error(`${cwd}/.gitlab-ci.yml is empty`);
+            process.exit(1);
+        }
+        orderedVariables.push(orderedYml.last().variables);
 
         // Parse .gitlab-ci.local.yml
-        orderedYml.push(yaml.safeLoad(fs.readFileSync(gitlabCiLocalYmlPath, "utf8")));
-        orderedVariables.push(orderedYml[orderedYml.length - 1].variables || {});
+        orderedYml.push(yaml.safeLoad(fs.readFileSync(gitlabCiLocalYmlPath, "utf8")) || {});
+        orderedVariables.push(orderedYml.last().variables || {});
 
         // Parse yamls included by other ci files.
         const includes = deepExtend.apply(this, orderedYml).include || [];
@@ -50,7 +54,7 @@ export class Parser {
             }
 
             orderedYml.unshift(yaml.safeLoad(fs.readFileSync(`${cwd}/${value.local}`, "utf8")));
-            orderedVariables.unshift(orderedYml[0].variables || {});
+            orderedVariables.unshift(orderedYml.first().variables || {});
         }
 
         // Setup variables and "merged" yml