Add option to disable linting on save

CHANGELOG.md

@@ -18,3 +18,5 @@ equivalent to 0.2.0
 - Initial release
 
 ## [Unreleased]
+
+- Added: `clang-tidy.lintOnSave` option to disable automatic linting on file save

README.md

@@ -26,6 +26,7 @@ This extension contributes the following settings:
 * `clang-tidy.checks`: List of checks to enable or disable
 * `clang-tidy.compilerArgs`: List of arguments to append to the compiler command line
 * `clang-tidy.compilerArgsBefore`: List of arguments to prepend to the compiler command line
+* `clang-tidy.lintOnSave`: Whether or not to lint files when they are saved
 
 ## Extension Commands
 

package.json

@@ -48,6 +48,11 @@
                     "type": "array",
                     "default": [],
                     "description": "Extra arguments to prepend to the compiler command line"
+                },
+                "clang-tidy.lintOnSave": {
+                    "type": "boolean",
+                    "default": true,
+                    "description": "Automatically lint files when they are saved"
                 }
             }
         }

src/extension.ts

@@ -24,13 +24,19 @@ export function activate(context: vscode.ExtensionContext) {
         }
     }
 
-    subscriptions.push(workspace.onDidSaveTextDocument(lintAndSetDiagnostics));
+    subscriptions.push(workspace.onDidSaveTextDocument(doc => {
+        if (workspace.getConfiguration('clang-tidy').get('lintOnSave')) {
+            lintAndSetDiagnostics(doc);
+        }
+    }));
     subscriptions.push(workspace.onDidOpenTextDocument(lintAndSetDiagnostics));
     subscriptions.push(workspace.onDidCloseTextDocument(doc => diagnosticCollection.delete(doc.uri)));
 
     subscriptions.push(workspace.onDidSaveTextDocument(doc => {
-        if (doc.uri.scheme === 'file' && doc.uri.fsPath.endsWith('.clang-tidy')) {
-            workspace.textDocuments.forEach(lintAndSetDiagnostics);
+        if (workspace.getConfiguration('clang-tidy').get('lintOnSave')) {
+            if (doc.uri.scheme === 'file' && doc.uri.fsPath.endsWith('.clang-tidy')) {
+                workspace.textDocuments.forEach(lintAndSetDiagnostics);
+            }
         }
     }));