Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(403)

Unified Diff: docs/vscode.md

Issue 2847663004: vscode.md: change watcherExlude out/ to out*/ to filter ChromeOS build output (Closed)
Patch Set: add some comments Created 3 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: docs/vscode.md
diff --git a/docs/vscode.md b/docs/vscode.md
index 65fd471035fccffe0b06e5d8d5f9b02ab53ed0c4..100be18bb4b7ce23163a1c825e971e6aee2cd900 100644
--- a/docs/vscode.md
+++ b/docs/vscode.md
@@ -192,10 +192,13 @@ Remember to replace `<full_path_to_your_home>`!
},
"files.watcherExclude": {
- // Don't check out/ and third_party/ for changes to fix issue
+ // Don't watch out*/ and third_party/ for changes to fix an issue
// where vscode doesn't notice that files have changed.
// https://github.com/Microsoft/vscode/issues/3998
- "**/out/**": true,
+ // There is currently another issue that requires a leading **/ for
+ // watcherExlude. Beware that this pattern might affect other out* folders
+ // like src/cc/output/.
+ "**/out*/**": true,
"**/third_party/**": true
},
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698