Merge pull request #5 from SanjulaGanepola/fix/step-exit-code

Call `process.exit` with step exit code and filter env vars with spaces

Author: worksofliam

.env.sample

@@ -0,0 +1,4 @@
+IBMI_HOST=
+IBMI_USER=
+IBMI_PASSWORD=
+IBMI_SSH_PORT=

.npmignore

@@ -5,4 +5,5 @@ test
 src
 !dist/src
 node_modules
-.vscode
+.vscode
+.env.sample

.vscode/launch.json

@@ -8,10 +8,10 @@
 			"name": "Debug ibmi-ci CLI",
 			"type": "node",
 			"request": "launch",
-			"program": "${workspaceFolder:ibmicli}/dist/index.js",
+			"program": "${workspaceFolder:ibmi-ci}/dist/index.js",
 			"sourceMaps": true,
 			"args": [
-				"--lcwd", "${workspaceFolder:ibmicli}/src", 
+				"--lcwd", "${workspaceFolder:ibmi-ci}/src", 
 				"--cmd", "mkdir -p './builds/icisrc'", 
 				"--ignore", "--cmd", "woejhraljdsfn",
 				"--rcwd", "./builds/icisrc", 
@@ -27,13 +27,13 @@
 				"type": "npm",
 				"script": "webpack:dev"
 			},
-			"envFile": "${workspaceFolder:ibmicli}/.env"
+			"envFile": "${workspaceFolder:ibmi-ci}/.env"
 		},
 		{
 			"name": "Debug ibmi-ci yaml",
 			"type": "node",
 			"request": "launch",
-			"program": "${workspaceFolder:ibmicli}/dist/index.js",
+			"program": "${workspaceFolder:ibmi-ci}/dist/index.js",
 			"sourceMaps": true,
 			"args": [
 				"--file", "examples/exampleA.yml",
@@ -42,7 +42,7 @@
 				"type": "npm",
 				"script": "webpack:dev"
 			},
-			"envFile": "${workspaceFolder:ibmicli}/.env"
+			"envFile": "${workspaceFolder:ibmi-ci}/.env"
 		},
   ]
 }

src/index.ts

@@ -72,6 +72,8 @@ async function main() {
 
 		const result = await executor.executeSteps({log: logger});
 		executor.dispose();
+		console.log(`ibmi-ci completed with exit code ${result.code}.`);
+		process.exit(result.code);
 	}
 }
 

src/steps/actions/env.ts

@@ -12,7 +12,7 @@ export class EnvironmentStep extends StepI {
       `CommonProgramFiles(x86)`, `ProgramFiles(x86)`
     ];
 
-    const environmentVariables = Object.keys(process.env).filter(key => !ignoredEnvironmentVariables.includes(key) && !key.includes('.'));
+    const environmentVariables = Object.keys(process.env).filter(key => !ignoredEnvironmentVariables.includes(key) && !key.includes('.') && !key.includes(' '));
     const commandString = environmentVariables.map(key => `${key}='${process.env[key]}'`).join(` `);
 
     this.log(`Setting environment variables: ${environmentVariables.join(`, `)}`);