Merge #104

104: Add vscode extension to CI r=aochagavia a=DJMcNab

Note that this testing is only done on travis - we are only running formatting and linting, so feature parity on appveyor is not required.

CC @aochagavia.

Fixes? #100

Co-authored-by: Daniel McNab <[email protected]>

2 files changed, 6 insertions, 9 deletions

diff --git a/editors/code/src/notifications/index.ts b/editors/code/src/notifications/index.ts
index c56576865..74c4c3563 100644
--- a/editors/code/src/notifications/index.ts
+++ b/editors/code/src/notifications/index.ts
@@ -1,5 +1,3 @@
 import * as publishDecorations from './publish_decorations';
 
-export {
-    publishDecorations,
-};
+export { publishDecorations };
diff --git a/editors/code/src/notifications/publish_decorations.ts b/editors/code/src/notifications/publish_decorations.ts
index d8790386b..3180019b7 100644
--- a/editors/code/src/notifications/publish_decorations.ts
+++ b/editors/code/src/notifications/publish_decorations.ts
@@ -10,11 +10,10 @@ export interface PublishDecorationsParams {
 
 export function handle(params: PublishDecorationsParams) {
     const targetEditor = vscode.window.visibleTextEditors.find(
-        (editor) => editor.document.uri.toString() === params.uri,
-    );
-    if (!Server.config.highlightingOn || !targetEditor) { return; }
-    Server.highlighter.setHighlights(
-        targetEditor,
-        params.decorations,
+        editor => editor.document.uri.toString() === params.uri
     );
+    if (!Server.config.highlightingOn || !targetEditor) {
+        return;
+    }
+    Server.highlighter.setHighlights(targetEditor, params.decorations);
 }