Add "Include" to extension TSConfig Files (microsoft#25860)

mjbvz · web-flow · commit 7d2dea4e454e · 2017-05-05T11:37:59.000-07:00
**Bug**
Most VSCode extensions currently specify `"exclude"` in their `tsconfig.json` but not `"include"`. This may result in extra files being included in each project

**Fix**
Add  `"include": ["src/**/*"]` to all extension tsconfig files
diff --git a/extensions/configuration-editing/tsconfig.json b/extensions/configuration-editing/tsconfig.json
@@ -8,7 +8,7 @@
 		],
 		"strictNullChecks": true
 	},
-	"exclude": [
-		"node_modules"
+	"include": [
+		"src/**/*"
 	]
 }
diff --git a/extensions/css/client/tsconfig.json b/extensions/css/client/tsconfig.json
@@ -7,7 +7,7 @@
 			"es5", "es2015.promise"
 		]
 	},
-	"exclude": [
-		"node_modules"
+	"include": [
+		"src/**/*"
 	]
 }
diff --git a/extensions/css/server/tsconfig.json b/extensions/css/server/tsconfig.json
@@ -7,7 +7,7 @@
 			"es5"
 		]
 	},
-	"exclude": [
-		"node_modules"
+	"include": [
+		"src/**/*"
 	]
 }
diff --git a/extensions/extension-editing/tsconfig.json b/extensions/extension-editing/tsconfig.json
@@ -7,7 +7,7 @@
 		"module": "commonjs",
 		"outDir": "./out"
 	},
-	"exclude": [
-		"node_modules"
+	"include": [
+		"src/**/*"
 	]
 }
diff --git a/extensions/git/tsconfig.json b/extensions/git/tsconfig.json
@@ -9,7 +9,7 @@
 		"strictNullChecks": true,
 		"experimentalDecorators": true
 	},
-	"exclude": [
-		"node_modules"
+	"include": [
+		"src/**/*"
 	]
 }
diff --git a/extensions/grunt/tsconfig.json b/extensions/grunt/tsconfig.json
@@ -12,8 +12,7 @@
 		"noUnusedLocals": true,
 		"noUnusedParameters": true
 	},
-	"exclude": [
-		"node_modules",
-		"out"
+	"include": [
+		"src/**/*"
 	]
 }
diff --git a/extensions/gulp/tsconfig.json b/extensions/gulp/tsconfig.json
@@ -12,8 +12,7 @@
 		"noUnusedLocals": true,
 		"noUnusedParameters": true
 	},
-	"exclude": [
-		"node_modules",
-		"out"
+	"include": [
+		"src/**/*"
 	]
 }
diff --git a/extensions/javascript/tsconfig.json b/extensions/javascript/tsconfig.json
@@ -7,7 +7,7 @@
 			"es2015"
 		]
 	},
-	"exclude": [
-		"node_modules"
+	"include": [
+		"src/**/*"
 	]
 }
diff --git a/extensions/json/server/tsconfig.json b/extensions/json/server/tsconfig.json
@@ -9,7 +9,7 @@
 			"es5", "es2015.promise"
 		]
 	},
-	"exclude": [
-		"node_modules"
+	"include": [
+		"src/**/*"
 	]
 }
diff --git a/extensions/markdown/tsconfig.json b/extensions/markdown/tsconfig.json
@@ -14,7 +14,7 @@
 		"noUnusedLocals": true,
 		"noUnusedParameters": true
 	},
-	"exclude": [
-		"node_modules"
+	"include": [
+		"src/**/*"
 	]
 }
diff --git a/extensions/php/tsconfig.json b/extensions/php/tsconfig.json
@@ -7,7 +7,7 @@
 		"module": "commonjs",
 		"outDir": "./out"
 	},
-	"exclude": [
-		"node_modules"
+	"include": [
+		"src/**/*"
 	]
 }
diff --git a/extensions/python/tsconfig.json b/extensions/python/tsconfig.json
@@ -7,7 +7,7 @@
 		"module": "commonjs",
 		"outDir": "./out"
 	},
-	"exclude": [
-		"node_modules"
+	"include": [
+		"src/**/*"
 	]
 }
diff --git a/extensions/ruby/tsconfig.json b/extensions/ruby/tsconfig.json
@@ -8,7 +8,7 @@
 		],
 		"sourceMap": true
 	},
-	"exclude": [
-		"node_modules"
+	"include": [
+		"src/**/*"
 	]
 }
diff --git a/extensions/typescript/tsconfig.json b/extensions/typescript/tsconfig.json
@@ -13,10 +13,7 @@
 		"noUnusedLocals": true,
 		"noUnusedParameters": true
 	},
-	"exclude": [
-		"node_modules",
-		"server",
-		"out",
-		"test/colorize-fixtures"
+	"include": [
+		"src/**/*"
 	]
 }
diff --git a/extensions/vscode-api-tests/tsconfig.json b/extensions/vscode-api-tests/tsconfig.json
@@ -9,7 +9,7 @@
 		"sourceMap": true,
 		"strictNullChecks": true
 	},
-	"exclude": [
-		"node_modules"
+	"include": [
+		"src/**/*"
 	]
 }
diff --git a/extensions/vscode-colorize-tests/tsconfig.json b/extensions/vscode-colorize-tests/tsconfig.json
@@ -8,7 +8,7 @@
 		],
 		"sourceMap": true
 	},
-	"exclude": [
-		"node_modules"
+	"include": [
+		"src/**/*"
 	]
 }

Original file line number	Diff line number	Diff line change
`@@ -8,7 +8,7 @@`
`8`	`8`	`],`
`9`	`9`	`"strictNullChecks": true`
`10`	`10`	`},`
`11`		`- "exclude": [`
`12`		`- "node_modules"`
	`11`	`+ "include": [`
	`12`	`+ "src/*/"`
`13`	`13`	`]`
`14`	`14`	`}`
Original file line number	Diff line number	Diff line change
`@@ -7,7 +7,7 @@`
`7`	`7`	`"es5", "es2015.promise"`
`8`	`8`	`]`
`9`	`9`	`},`
`10`		`- "exclude": [`
`11`		`- "node_modules"`
	`10`	`+ "include": [`
	`11`	`+ "src/*/"`
`12`	`12`	`]`
`13`	`13`	`}`
Original file line number	Diff line number	Diff line change
`@@ -7,7 +7,7 @@`
`7`	`7`	`"es5"`
`8`	`8`	`]`
`9`	`9`	`},`
`10`		`- "exclude": [`
`11`		`- "node_modules"`
	`10`	`+ "include": [`
	`11`	`+ "src/*/"`
`12`	`12`	`]`
`13`	`13`	`}`
Original file line number	Diff line number	Diff line change
`@@ -7,7 +7,7 @@`
`7`	`7`	`"module": "commonjs",`
`8`	`8`	`"outDir": "./out"`
`9`	`9`	`},`
`10`		`- "exclude": [`
`11`		`- "node_modules"`
	`10`	`+ "include": [`
	`11`	`+ "src/*/"`
`12`	`12`	`]`
`13`	`13`	`}`
Original file line number	Diff line number	Diff line change
`@@ -9,7 +9,7 @@`
`9`	`9`	`"strictNullChecks": true,`
`10`	`10`	`"experimentalDecorators": true`
`11`	`11`	`},`
`12`		`- "exclude": [`
`13`		`- "node_modules"`
	`12`	`+ "include": [`
	`13`	`+ "src/*/"`
`14`	`14`	`]`
`15`	`15`	`}`
Original file line number	Diff line number	Diff line change
`@@ -12,8 +12,7 @@`
`12`	`12`	`"noUnusedLocals": true,`
`13`	`13`	`"noUnusedParameters": true`
`14`	`14`	`},`
`15`		`- "exclude": [`
`16`		`- "node_modules",`
`17`		`- "out"`
	`15`	`+ "include": [`
	`16`	`+ "src/*/"`
`18`	`17`	`]`
`19`	`18`	`}`
Original file line number	Diff line number	Diff line change
`@@ -7,7 +7,7 @@`
`7`	`7`	`"es2015"`
`8`	`8`	`]`
`9`	`9`	`},`
`10`		`- "exclude": [`
`11`		`- "node_modules"`
	`10`	`+ "include": [`
	`11`	`+ "src/*/"`
`12`	`12`	`]`
`13`	`13`	`}`
Original file line number	Diff line number	Diff line change
`@@ -14,7 +14,7 @@`
`14`	`14`	`"noUnusedLocals": true,`
`15`	`15`	`"noUnusedParameters": true`
`16`	`16`	`},`
`17`		`- "exclude": [`
`18`		`- "node_modules"`
	`17`	`+ "include": [`
	`18`	`+ "src/*/"`
`19`	`19`	`]`
`20`	`20`	`}`