Merge pull request #3 from bmalehorn/hide-codeowners

hide status bar item when no CODEOWNERS file
jasonnutter · Jul 25, 2019 · e5dfe2c · e5dfe2c
2 parents 71838dc + 6495dd7
commit e5dfe2c
Showing 1 changed file with 12 additions and 1 deletion.
diff --git a/extension.js b/extension.js
@@ -15,7 +15,13 @@ const getOwners = () => {
         uri: { fsPath: workspacePath }
     } = vscode.workspace.getWorkspaceFolder(uri);
 
-    const folder = new Codeowners(workspacePath);
+    let folder;
+    try {
+        folder = new Codeowners(workspacePath);
+    } catch {
+        // no CODEOWNERS file
+        return null;
+    }
 
     const file = fileName.split(workspacePath)[1];
 
@@ -40,6 +46,11 @@ const activate = context => {
         vscode.window.onDidChangeActiveTextEditor(() => {
             const owners = getOwners();
 
+            if (!owners) {
+                statusBarItem.hide();
+                return;
+            }
+
             if (owners.length > 2) {
                 statusBarItem.text = `CODEOWNERS: ${owners[0]} & ${owners.length - 1} others`;
             } else if (owners.length === 2) {