Merge pull request #176 from matrix-org/hs/fix-confusing-irc-message

Fix confusing irc message

Author: Half-Shot

src/commands/IrcPlumbCommand.ts

@@ -66,16 +66,18 @@ export class IrcPlumbCommand implements ICommand {
             await this.ircBridge.plumbChannelToRoom(channel, resolvedRoomId);
         } catch (ex) {
             LogService.warn("IrcPlumbCommand", ex);
-            return await logMessage(LogLevel.WARN, "IrcPlumbCommand", `Could not plumb channel to room ${resolvedRoomId}`);
+            return logMessage(LogLevel.WARN, "IrcPlumbCommand", `Could not plumb channel to room ${resolvedRoomId}`);
         }
 
         try {
             // The bridge needs the ability to kick KLINED users.
             await client.setUserPowerLevel(this.ircBridge.botUserId, resolvedRoomId, KickPowerLevel);
         } catch (ex) {
             LogService.warn("IrcPlumbCommand", ex);
-            return await logMessage(LogLevel.WARN, "IrcPlumbCommand", `Could not plumb channel to room ${resolvedRoomId}: could not set AS power level`);
+            return logMessage(LogLevel.WARN, "IrcPlumbCommand", `Could not plumb channel to room ${resolvedRoomId}: could not set AS power level`);
         }
+
+        logMessage(LogLevel.INFO,"IrcPlumbCommand", `Plumbed channel ${channel} to ${resolvedRoomId}`);
     }
 
     public async run(conference: Conference, client: MatrixClient, roomId: string, event: any, args: string[]) {
@@ -105,8 +107,6 @@ export class IrcPlumbCommand implements ICommand {
             return client.sendNotice(roomId, "Could not join that room, is the bot invited?");
         }
 
-        await this.plumbOne(client, resolvedRoomId, channel);
-
-        return client.sendNotice(roomId, "Plumbed channel");
+        return this.plumbOne(client, resolvedRoomId, channel);
     }
 }