Add files.watcherExlude to vscode.md

docs/vscode.md

Index: docs/vscode.md
diff --git a/docs/vscode.md b/docs/vscode.md
index 7418aa02eabae27defba3308735fba4fd91f547e..65fd471035fccffe0b06e5d8d5f9b02ab53ed0c4 100644
--- a/docs/vscode.md
+++ b/docs/vscode.md
@@ -191,6 +191,14 @@ Remember to replace `<full_path_to_your_home>`!
     "out*/**": true
   },
 
+  "files.watcherExclude": {

[ljusten (tachyonic), 2017/04/27 14:16:24]

Out of curiosity, did that cause any issues? Did you notice any improvement?

[dullweber, 2017/04/27 14:27:29]

I had the problem that when I run git cl format, vscode didn't notice any
changes to the currently open file. Adding watcherExclude fixed this.

[ljusten (tachyonic), 2017/04/27 15:54:42]

Acknowledged.

+    // Don't check out/ and third_party/ for changes to fix issue
+    // where vscode doesn't notice that files have changed.
+    // https://github.com/Microsoft/vscode/issues/3998
+    "**/out/**": true,

[ljusten (tachyonic), 2017/04/27 14:16:24]

Suggest to use **/out*/** since IIRC some Chrome OS workflows generate
out-$board directories.

Is excluding out really necessary given that it's already excluded by
files.exclude?

[dullweber, 2017/04/27 14:27:29]

files.exlude is not applied to the watcher for some reason. Maybe the issue only
hit me because I have quite a few different directories in out/ with different
configurations. 
I guess it is mostly important to reduce the number of files that are being
watched. Unless these chromeos folders also contains hundrets of thousands of
files it probably doesn't matter.

[ljusten (tachyonic), 2017/04/27 15:54:42]

The chromeos folders are regular build folders, so yes, they're big.

+    "**/third_party/**": true

[ljusten (tachyonic), 2017/04/27 14:16:24]

The initial **/ seems unnecessary (files.excludes works without that), but
according to jsturgis' comment on the github issue it appears to be required.
Maybe add a comment?

[dullweber, 2017/04/27 14:27:29]

It is currently required to have the initial **/, I guess that is a bug but it
doesn't work without this. I subscribed to the github issue and will remove the
**/ when it is fixed.

[ljusten (tachyonic), 2017/04/27 15:54:42]

Acknowledged.

+  },
+
   // Wider author column for annotator extension.
   "annotator.annotationColumnWidth": "24em",