fix(lint): fix noImplicitReturns TS issues

pact-foundation · Nov 19, 2017 · 5c57288 · 5c57288
1 parent d4f8e7b
commit 5c57288
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 4 deletions.
diff --git a/src/server.ts b/src/server.ts
@@ -149,7 +149,7 @@ export class Server extends events.EventEmitter {
 	public start(): q.Promise<Server> {
 		if (this.__instance && this.__instance.connected) {
 			logger.warn(`You already have a process running with PID: ${this.__instance.pid}`);
-			return;
+			return null;
 		}
 		this.__instance = pactUtil.spawnBinary(`${pact.mockServicePath} service`, this.options, this.__argMapping);
 		this.__instance.once("close", () => this.stop());

diff --git a/test/integration/data-utils.ts b/test/integration/data-utils.ts
@@ -13,7 +13,7 @@ export function auth(req: express.Request, res: express.Response, next: express.
 	const user = basicAuth(req);
 	if (user && user.name === "foo" && user.pass === "bar") {
 		next();
-		return;
+		return null;
 	}
 
 	res.set("WWW-Authenticate", "Basic realm=Authorization Required");

diff --git a/tsconfig.json b/tsconfig.json
@@ -2,7 +2,6 @@
 	"compilerOptions": {
 		"module": "commonjs",
 		"target": "es5",
-		"noImplicitAny": false,
 		"removeComments": true,
 		"noLib": false,
 		"emitDecoratorMetadata": true,
@@ -12,7 +11,8 @@
 		"isolatedModules": false,
 		"moduleResolution": "node",
 		"suppressImplicitAnyIndexErrors": true,
-		"declaration": true
+		"declaration": true,
+		"noImplicitReturns": true
 	},
 	"include": [
 		"**/*.ts",