-
Notifications
You must be signed in to change notification settings - Fork 12.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add link to documentation in tsconfig.json template generated by tsc --init #34686
Changes from 6 commits
ac455d8
99cc043
b4b64c5
440ac98
80d8e40
f166f9d
e0a0d65
a6d53ae
6ef2ba2
28c0040
e8196f9
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -150,6 +150,14 @@ namespace ts { | |
category: Diagnostics.Command_line_Options, | ||
description: Diagnostics.Whether_to_keep_outdated_console_output_in_watch_mode_instead_of_clearing_the_screen, | ||
}, | ||
{ | ||
name: "incremental", | ||
shortName: "i", | ||
type: "boolean", | ||
category: Diagnostics.Basic_Options, | ||
description: Diagnostics.Enable_incremental_compilation, | ||
transpileOptionValue: undefined | ||
}, | ||
{ | ||
name: "listFiles", | ||
type: "boolean", | ||
|
@@ -2108,10 +2116,9 @@ namespace ts { | |
|
||
function isAllowedOption({ category, name }: CommandLineOption): boolean { | ||
// Skip options which do not have a category or have category `Command_line_Options` | ||
// Exclude all possible `Advanced_Options` in tsconfig.json which were NOT defined in command line | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Return the comment, remove the extra There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. done There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. The comment is still gone. Can you please fix that so that its ready to merge |
||
return category !== undefined | ||
&& category !== Diagnostics.Command_line_Options | ||
&& (category !== Diagnostics.Advanced_Options || compilerOptionsMap.has(name)); | ||
&& (category !== Diagnostics.Advanced_Options || compilerOptionsMap.has(name));; | ||
} | ||
|
||
function writeConfigurations() { | ||
|
@@ -2129,14 +2136,11 @@ namespace ts { | |
let marginLength = 0; | ||
let seenKnownKeys = 0; | ||
const nameColumn: string[] = []; | ||
const descriptionColumn: string[] = []; | ||
categorizedOptions.forEach((options, category) => { | ||
if (nameColumn.length !== 0) { | ||
nameColumn.push(""); | ||
descriptionColumn.push(""); | ||
} | ||
nameColumn.push(`/* ${category} */`); | ||
descriptionColumn.push(""); | ||
for (const option of options) { | ||
let optionName; | ||
if (compilerOptionsMap.has(option.name)) { | ||
|
@@ -2146,7 +2150,6 @@ namespace ts { | |
optionName = `// "${option.name}": ${JSON.stringify(getDefaultValueForOption(option))},`; | ||
} | ||
nameColumn.push(optionName); | ||
descriptionColumn.push(`/* ${option.description && getLocaleSpecificMessage(option.description) || option.name} */`); | ||
marginLength = Math.max(optionName.length, marginLength); | ||
} | ||
}); | ||
|
@@ -2156,11 +2159,11 @@ namespace ts { | |
const result: string[] = []; | ||
result.push(`{`); | ||
result.push(`${tab}"compilerOptions": {`); | ||
result.push(`${tab}${tab}/* ${getLocaleSpecificMessage(Diagnostics.Visit_https_Colon_Slash_Slashaka_ms_Slashtsconfig_json_to_read_more_about_this_file)} */`); | ||
result.push(""); | ||
// Print out each row, aligning all the descriptions on the same column. | ||
for (let i = 0; i < nameColumn.length; i++) { | ||
const optionName = nameColumn[i]; | ||
const description = descriptionColumn[i]; | ||
result.push(optionName && `${tab}${tab}${optionName}${description && (makePadding(marginLength - optionName.length + 2) + description)}`); | ||
for (const optionName of nameColumn) { | ||
result.push(optionName && `${tab}${tab}${optionName}`); | ||
} | ||
if (fileNames.length) { | ||
result.push(`${tab}},`); | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,66 +1,69 @@ | ||
{ | ||
"compilerOptions": { | ||
/* Visit https://aka.ms/tsconfig.json to read more about this file */ | ||
|
||
/* Basic Options */ | ||
// "incremental": true, /* Enable incremental compilation */ | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Example shows the comments as not removed #37361 (comment) @RyanCavanaugh @DanielRosenwasser Whats the final call on that |
||
"target": "es5", /* Specify ECMAScript target version: 'ES3' (default), 'ES5', 'ES2015', 'ES2016', 'ES2017', 'ES2018', 'ES2019', 'ES2020', or 'ESNEXT'. */ | ||
"module": "commonjs", /* Specify module code generation: 'none', 'commonjs', 'amd', 'system', 'umd', 'es2015', 'es2020', or 'ESNext'. */ | ||
// "lib": [], /* Specify library files to be included in the compilation. */ | ||
// "allowJs": true, /* Allow javascript files to be compiled. */ | ||
// "checkJs": true, /* Report errors in .js files. */ | ||
// "jsx": "preserve", /* Specify JSX code generation: 'preserve', 'react-native', or 'react'. */ | ||
// "declaration": true, /* Generates corresponding '.d.ts' file. */ | ||
// "declarationMap": true, /* Generates a sourcemap for each corresponding '.d.ts' file. */ | ||
// "sourceMap": true, /* Generates corresponding '.map' file. */ | ||
// "outFile": "./", /* Concatenate and emit output to single file. */ | ||
// "outDir": "./", /* Redirect output structure to the directory. */ | ||
// "rootDir": "./", /* Specify the root directory of input files. Use to control the output directory structure with --outDir. */ | ||
// "composite": true, /* Enable project compilation */ | ||
// "tsBuildInfoFile": "./", /* Specify file to store incremental compilation information */ | ||
// "removeComments": true, /* Do not emit comments to output. */ | ||
// "noEmit": true, /* Do not emit outputs. */ | ||
// "importHelpers": true, /* Import emit helpers from 'tslib'. */ | ||
// "downlevelIteration": true, /* Provide full support for iterables in 'for-of', spread, and destructuring when targeting 'ES5' or 'ES3'. */ | ||
// "isolatedModules": true, /* Transpile each file as a separate module (similar to 'ts.transpileModule'). */ | ||
// "incremental": true, | ||
// "incremental": true, | ||
"target": "es5", | ||
"module": "commonjs", | ||
// "lib": [], | ||
// "allowJs": true, | ||
// "checkJs": true, | ||
// "jsx": "preserve", | ||
// "declaration": true, | ||
// "declarationMap": true, | ||
// "sourceMap": true, | ||
// "outFile": "./", | ||
// "outDir": "./", | ||
// "rootDir": "./", | ||
// "composite": true, | ||
// "tsBuildInfoFile": "./", | ||
// "removeComments": true, | ||
// "noEmit": true, | ||
// "importHelpers": true, | ||
// "downlevelIteration": true, | ||
// "isolatedModules": true, | ||
|
||
/* Strict Type-Checking Options */ | ||
"strict": true, /* Enable all strict type-checking options. */ | ||
// "noImplicitAny": true, /* Raise error on expressions and declarations with an implied 'any' type. */ | ||
// "strictNullChecks": true, /* Enable strict null checks. */ | ||
// "strictFunctionTypes": true, /* Enable strict checking of function types. */ | ||
// "strictBindCallApply": true, /* Enable strict 'bind', 'call', and 'apply' methods on functions. */ | ||
// "strictPropertyInitialization": true, /* Enable strict checking of property initialization in classes. */ | ||
// "noImplicitThis": true, /* Raise error on 'this' expressions with an implied 'any' type. */ | ||
// "alwaysStrict": true, /* Parse in strict mode and emit "use strict" for each source file. */ | ||
"strict": true, | ||
// "noImplicitAny": true, | ||
// "strictNullChecks": true, | ||
// "strictFunctionTypes": true, | ||
// "strictBindCallApply": true, | ||
// "strictPropertyInitialization": true, | ||
// "noImplicitThis": true, | ||
// "alwaysStrict": true, | ||
|
||
/* Additional Checks */ | ||
// "noUnusedLocals": true, /* Report errors on unused locals. */ | ||
// "noUnusedParameters": true, /* Report errors on unused parameters. */ | ||
// "noImplicitReturns": true, /* Report error when not all code paths in function return a value. */ | ||
// "noFallthroughCasesInSwitch": true, /* Report errors for fallthrough cases in switch statement. */ | ||
// "noUnusedLocals": true, | ||
// "noUnusedParameters": true, | ||
// "noImplicitReturns": true, | ||
// "noFallthroughCasesInSwitch": true, | ||
|
||
/* Module Resolution Options */ | ||
// "moduleResolution": "node", /* Specify module resolution strategy: 'node' (Node.js) or 'classic' (TypeScript pre-1.6). */ | ||
// "baseUrl": "./", /* Base directory to resolve non-absolute module names. */ | ||
// "paths": {}, /* A series of entries which re-map imports to lookup locations relative to the 'baseUrl'. */ | ||
// "rootDirs": [], /* List of root folders whose combined content represents the structure of the project at runtime. */ | ||
// "typeRoots": [], /* List of folders to include type definitions from. */ | ||
// "types": [], /* Type declaration files to be included in compilation. */ | ||
// "allowSyntheticDefaultImports": true, /* Allow default imports from modules with no default export. This does not affect code emit, just typechecking. */ | ||
"esModuleInterop": true, /* Enables emit interoperability between CommonJS and ES Modules via creation of namespace objects for all imports. Implies 'allowSyntheticDefaultImports'. */ | ||
// "preserveSymlinks": true, /* Do not resolve the real path of symlinks. */ | ||
// "allowUmdGlobalAccess": true, /* Allow accessing UMD globals from modules. */ | ||
// "moduleResolution": "node", | ||
// "baseUrl": "./", | ||
// "paths": {}, | ||
// "rootDirs": [], | ||
// "typeRoots": [], | ||
// "types": [], | ||
// "allowSyntheticDefaultImports": true, | ||
"esModuleInterop": true, | ||
// "preserveSymlinks": true, | ||
// "allowUmdGlobalAccess": true, | ||
|
||
/* Source Map Options */ | ||
// "sourceRoot": "", /* Specify the location where debugger should locate TypeScript files instead of source locations. */ | ||
// "mapRoot": "", /* Specify the location where debugger should locate map files instead of generated locations. */ | ||
// "inlineSourceMap": true, /* Emit a single file with source maps instead of having a separate file. */ | ||
// "inlineSources": true, /* Emit the source alongside the sourcemaps within a single file; requires '--inlineSourceMap' or '--sourceMap' to be set. */ | ||
// "sourceRoot": "", | ||
// "mapRoot": "", | ||
// "inlineSourceMap": true, | ||
// "inlineSources": true, | ||
|
||
/* Experimental Options */ | ||
// "experimentalDecorators": true, /* Enables experimental support for ES7 decorators. */ | ||
// "emitDecoratorMetadata": true, /* Enables experimental support for emitting type metadata for decorators. */ | ||
// "experimentalDecorators": true, | ||
// "emitDecoratorMetadata": true, | ||
|
||
/* Advanced Options */ | ||
"forceConsistentCasingInFileNames": true /* Disallow inconsistently-cased references to the same file. */ | ||
"forceConsistentCasingInFileNames": true | ||
} | ||
} |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,69 +1,72 @@ | ||
{ | ||
"compilerOptions": { | ||
/* Visit https://aka.ms/tsconfig.json to read more about this file */ | ||
|
||
/* Basic Options */ | ||
// "incremental": true, /* Enable incremental compilation */ | ||
"target": "es5", /* Specify ECMAScript target version: 'ES3' (default), 'ES5', 'ES2015', 'ES2016', 'ES2017', 'ES2018', 'ES2019', 'ES2020', or 'ESNEXT'. */ | ||
"module": "commonjs", /* Specify module code generation: 'none', 'commonjs', 'amd', 'system', 'umd', 'es2015', 'es2020', or 'ESNext'. */ | ||
// "lib": [], /* Specify library files to be included in the compilation. */ | ||
// "allowJs": true, /* Allow javascript files to be compiled. */ | ||
// "checkJs": true, /* Report errors in .js files. */ | ||
// "jsx": "preserve", /* Specify JSX code generation: 'preserve', 'react-native', or 'react'. */ | ||
"declaration": true, /* Generates corresponding '.d.ts' file. */ | ||
// "declarationMap": true, /* Generates a sourcemap for each corresponding '.d.ts' file. */ | ||
// "sourceMap": true, /* Generates corresponding '.map' file. */ | ||
// "outFile": "./", /* Concatenate and emit output to single file. */ | ||
// "outDir": "./", /* Redirect output structure to the directory. */ | ||
// "rootDir": "./", /* Specify the root directory of input files. Use to control the output directory structure with --outDir. */ | ||
// "composite": true, /* Enable project compilation */ | ||
// "tsBuildInfoFile": "./", /* Specify file to store incremental compilation information */ | ||
// "removeComments": true, /* Do not emit comments to output. */ | ||
// "noEmit": true, /* Do not emit outputs. */ | ||
// "importHelpers": true, /* Import emit helpers from 'tslib'. */ | ||
// "downlevelIteration": true, /* Provide full support for iterables in 'for-of', spread, and destructuring when targeting 'ES5' or 'ES3'. */ | ||
// "isolatedModules": true, /* Transpile each file as a separate module (similar to 'ts.transpileModule'). */ | ||
// "incremental": true, | ||
// "incremental": true, | ||
"target": "es5", | ||
"module": "commonjs", | ||
// "lib": [], | ||
// "allowJs": true, | ||
// "checkJs": true, | ||
// "jsx": "preserve", | ||
"declaration": true, | ||
// "declarationMap": true, | ||
// "sourceMap": true, | ||
// "outFile": "./", | ||
// "outDir": "./", | ||
// "rootDir": "./", | ||
// "composite": true, | ||
// "tsBuildInfoFile": "./", | ||
// "removeComments": true, | ||
// "noEmit": true, | ||
// "importHelpers": true, | ||
// "downlevelIteration": true, | ||
// "isolatedModules": true, | ||
|
||
/* Strict Type-Checking Options */ | ||
"strict": true, /* Enable all strict type-checking options. */ | ||
// "noImplicitAny": true, /* Raise error on expressions and declarations with an implied 'any' type. */ | ||
// "strictNullChecks": true, /* Enable strict null checks. */ | ||
// "strictFunctionTypes": true, /* Enable strict checking of function types. */ | ||
// "strictBindCallApply": true, /* Enable strict 'bind', 'call', and 'apply' methods on functions. */ | ||
// "strictPropertyInitialization": true, /* Enable strict checking of property initialization in classes. */ | ||
// "noImplicitThis": true, /* Raise error on 'this' expressions with an implied 'any' type. */ | ||
// "alwaysStrict": true, /* Parse in strict mode and emit "use strict" for each source file. */ | ||
"strict": true, | ||
// "noImplicitAny": true, | ||
// "strictNullChecks": true, | ||
// "strictFunctionTypes": true, | ||
// "strictBindCallApply": true, | ||
// "strictPropertyInitialization": true, | ||
// "noImplicitThis": true, | ||
// "alwaysStrict": true, | ||
|
||
/* Additional Checks */ | ||
// "noUnusedLocals": true, /* Report errors on unused locals. */ | ||
// "noUnusedParameters": true, /* Report errors on unused parameters. */ | ||
// "noImplicitReturns": true, /* Report error when not all code paths in function return a value. */ | ||
// "noFallthroughCasesInSwitch": true, /* Report errors for fallthrough cases in switch statement. */ | ||
// "noUnusedLocals": true, | ||
// "noUnusedParameters": true, | ||
// "noImplicitReturns": true, | ||
// "noFallthroughCasesInSwitch": true, | ||
|
||
/* Module Resolution Options */ | ||
// "moduleResolution": "node", /* Specify module resolution strategy: 'node' (Node.js) or 'classic' (TypeScript pre-1.6). */ | ||
// "baseUrl": "./", /* Base directory to resolve non-absolute module names. */ | ||
// "paths": {}, /* A series of entries which re-map imports to lookup locations relative to the 'baseUrl'. */ | ||
// "rootDirs": [], /* List of root folders whose combined content represents the structure of the project at runtime. */ | ||
// "typeRoots": [], /* List of folders to include type definitions from. */ | ||
// "types": [], /* Type declaration files to be included in compilation. */ | ||
// "allowSyntheticDefaultImports": true, /* Allow default imports from modules with no default export. This does not affect code emit, just typechecking. */ | ||
"esModuleInterop": true, /* Enables emit interoperability between CommonJS and ES Modules via creation of namespace objects for all imports. Implies 'allowSyntheticDefaultImports'. */ | ||
// "preserveSymlinks": true, /* Do not resolve the real path of symlinks. */ | ||
// "allowUmdGlobalAccess": true, /* Allow accessing UMD globals from modules. */ | ||
// "moduleResolution": "node", | ||
// "baseUrl": "./", | ||
// "paths": {}, | ||
// "rootDirs": [], | ||
// "typeRoots": [], | ||
// "types": [], | ||
// "allowSyntheticDefaultImports": true, | ||
"esModuleInterop": true, | ||
// "preserveSymlinks": true, | ||
// "allowUmdGlobalAccess": true, | ||
|
||
/* Source Map Options */ | ||
// "sourceRoot": "", /* Specify the location where debugger should locate TypeScript files instead of source locations. */ | ||
// "mapRoot": "", /* Specify the location where debugger should locate map files instead of generated locations. */ | ||
// "inlineSourceMap": true, /* Emit a single file with source maps instead of having a separate file. */ | ||
// "inlineSources": true, /* Emit the source alongside the sourcemaps within a single file; requires '--inlineSourceMap' or '--sourceMap' to be set. */ | ||
// "sourceRoot": "", | ||
// "mapRoot": "", | ||
// "inlineSourceMap": true, | ||
// "inlineSources": true, | ||
|
||
/* Experimental Options */ | ||
// "experimentalDecorators": true, /* Enables experimental support for ES7 decorators. */ | ||
// "emitDecoratorMetadata": true, /* Enables experimental support for emitting type metadata for decorators. */ | ||
// "experimentalDecorators": true, | ||
// "emitDecoratorMetadata": true, | ||
|
||
/* Advanced Options */ | ||
"noErrorTruncation": true, /* Do not truncate error messages. */ | ||
"declarationDir": "lib", /* Output directory for generated declaration files. */ | ||
"skipLibCheck": true, /* Skip type checking of declaration files. */ | ||
"forceConsistentCasingInFileNames": true /* Disallow inconsistently-cased references to the same file. */ | ||
"noErrorTruncation": true, | ||
"declarationDir": "lib", | ||
"skipLibCheck": true, | ||
"forceConsistentCasingInFileNames": true | ||
} | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There is already incremental included in this list why is there another entry?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
guess that was automerged and I missed it - pushed a fix for it
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you please existing entry instead? Is there reason this needs to move ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
better now?