refactor: Svn errors now show the repo name in the output channel (#514)

Author: JohnstonCode

src/model.ts

@@ -18,6 +18,9 @@ import {
   Status
 } from "./common/types";
 import { debounce } from "./decorators";
+import { exists } from "./fs/exists";
+import { readdir } from "./fs/readdir";
+import { stat } from "./fs/stat";
 import { configuration } from "./helpers/configuration";
 import { RemoteRepository } from "./remoteRepository";
 import { Repository } from "./repository";
@@ -31,9 +34,6 @@ import {
   isDescendant,
   normalizePath
 } from "./util";
-import { exists } from "./fs/exists";
-import { readdir } from "./fs/readdir";
-import { stat } from "./fs/stat";
 import { matchAll } from "./util/globMatch";
 
 export class Model implements IDisposable {

src/svn.ts

@@ -155,13 +155,15 @@ export class Svn {
       }),
       new Promise<Buffer>(resolve => {
         const buffers: Buffer[] = [];
-        on((process.stdout as Readable), "data", (b: Buffer) => buffers.push(b));
-        once((process.stdout as Readable), "close", () => resolve(Buffer.concat(buffers)));
+        on(process.stdout as Readable, "data", (b: Buffer) => buffers.push(b));
+        once(process.stdout as Readable, "close", () =>
+          resolve(Buffer.concat(buffers))
+        );
       }),
       new Promise<string>(resolve => {
         const buffers: Buffer[] = [];
-        on((process.stderr as Readable), "data", (b: Buffer) => buffers.push(b));
-        once((process.stderr as Readable), "close", () =>
+        on(process.stderr as Readable, "data", (b: Buffer) => buffers.push(b));
+        once(process.stderr as Readable, "close", () =>
           resolve(Buffer.concat(buffers).toString())
         );
       })
@@ -201,7 +203,13 @@ export class Svn {
     const decodedStdout = iconv.decode(stdout, encoding);
 
     if (options.log !== false && stderr.length > 0) {
-      this.logOutput(`${stderr}\n`);
+      const name = this.lastCwd.split(/[\\\/]+/).pop();
+      const err = stderr
+        .split("\n")
+        .filter((line: string) => line)
+        .map((line: string) => `[${name}]$ ${line}`)
+        .join("\n");
+      this.logOutput(err);
     }
 
     if (exitCode) {

src/svnContentProvider.ts

@@ -110,7 +110,7 @@ export class SvnContentProvider
         return await repository.patch([fsPath]);
       }
     } catch (error) {
-      console.error(error);
+      // Dont show error
     }
     return "";
   }