Skip to content

Add --preserveValueImports #44619

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.

Sign up for GitHub

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

Merged
andrewbranch merged 21 commits into from
Sep 8, 2021
Merged

Add --preserveValueImports #44619

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
21 commits
Select commit Hold shift + click to select a range
162ad55
Add compiler option
andrewbranch May 7, 2021
da43639
Require es2015+
andrewbranch May 7, 2021
a755070
Do not elide any imports or exports in preserve-exact
andrewbranch May 7, 2021
8dd2455
Add errors for writing imports/exports that reference elided names
andrewbranch May 17, 2021
e6df1e5
Improve diagnostics wording
andrewbranch May 18, 2021
126069d
Update API baselines
andrewbranch May 18, 2021
3063638
Redo as noEraslingImportedNames
andrewbranch Jun 16, 2021
fccdf1f
Merge branch 'main' into feature/noErasingImportedNames
andrewbranch Jun 16, 2021
30ffc15
Update option category
andrewbranch Jun 16, 2021
0b02873
Update baselines
andrewbranch Jun 16, 2021
bc850fa
Lint
andrewbranch Aug 18, 2021
71f4e71
Fix up transformer comments
andrewbranch Aug 18, 2021
961a014
Merge branch 'main' into feature/noErasingImportedNames
andrewbranch Aug 18, 2021
b5ed171
Fix errors from merge
andrewbranch Aug 18, 2021
bc91a25
Update other error code baseline
andrewbranch Aug 19, 2021
f8c6869
Rename to "preserveValueImports"
andrewbranch Aug 26, 2021
9e2e4dc
Clean up, reword diagnostics
andrewbranch Sep 8, 2021
bff0093
Update API baselines
andrewbranch Sep 8, 2021
dd5ab20
Update other baseline affected by error message reword
andrewbranch Sep 8, 2021
dc9e02d
Update tsconfig baselines
andrewbranch Sep 8, 2021
c27d59a
Add debug assertion instead of !
andrewbranch Sep 8, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Update baselines
  • Loading branch information
@andrewbranch
andrewbranch committed Jun 16, 2021
commit 0b02873d484c871dd28ac7f8e6272a2221e5c6ce
1 change: 1 addition & 0 deletions tests/baselines/reference/tsConfig/Default initialized TSConfig/tsconfig.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -64,6 +64,7 @@
// "noEmitOnError": true, /* Disable emitting files if any type checking errors are reported. */
// "preserveConstEnums": true, /* Disable erasing `const enum` declarations in generated code. */
// "declarationDir": "./", /* Specify the output directory for generated declaration files. */
// "noErasingImportedNames": true, /* Disable the removal of unused imported identifiers from the JavaScript output. */

/* Interop Constraints */
// "isolatedModules": true, /* Ensure that each file can be safely transpiled without relying on other imports. */
Expand Down
1 change: 1 addition & 0 deletions tests/baselines/reference/tsConfig/Initialized TSConfig with advanced options/tsconfig.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -64,6 +64,7 @@
// "noEmitOnError": true, /* Disable emitting files if any type checking errors are reported. */
// "preserveConstEnums": true, /* Disable erasing `const enum` declarations in generated code. */
"declarationDir": "lib", /* Specify the output directory for generated declaration files. */
// "noErasingImportedNames": true, /* Disable the removal of unused imported identifiers from the JavaScript output. */

/* Interop Constraints */
// "isolatedModules": true, /* Ensure that each file can be safely transpiled without relying on other imports. */
Expand Down
1 change: 1 addition & 0 deletions ...reference/tsConfig/Initialized TSConfig with boolean value compiler options/tsconfig.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -64,6 +64,7 @@
// "noEmitOnError": true, /* Disable emitting files if any type checking errors are reported. */
// "preserveConstEnums": true, /* Disable erasing `const enum` declarations in generated code. */
// "declarationDir": "./", /* Specify the output directory for generated declaration files. */
// "noErasingImportedNames": true, /* Disable the removal of unused imported identifiers from the JavaScript output. */

/* Interop Constraints */
// "isolatedModules": true, /* Ensure that each file can be safely transpiled without relying on other imports. */
Expand Down
1 change: 1 addition & 0 deletions ...es/reference/tsConfig/Initialized TSConfig with enum value compiler options/tsconfig.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -64,6 +64,7 @@
// "noEmitOnError": true, /* Disable emitting files if any type checking errors are reported. */
// "preserveConstEnums": true, /* Disable erasing `const enum` declarations in generated code. */
// "declarationDir": "./", /* Specify the output directory for generated declaration files. */
// "noErasingImportedNames": true, /* Disable the removal of unused imported identifiers from the JavaScript output. */

/* Interop Constraints */
// "isolatedModules": true, /* Ensure that each file can be safely transpiled without relying on other imports. */
Expand Down
1 change: 1 addition & 0 deletions tests/baselines/reference/tsConfig/Initialized TSConfig with files options/tsconfig.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -64,6 +64,7 @@
// "noEmitOnError": true, /* Disable emitting files if any type checking errors are reported. */
// "preserveConstEnums": true, /* Disable erasing `const enum` declarations in generated code. */
// "declarationDir": "./", /* Specify the output directory for generated declaration files. */
// "noErasingImportedNames": true, /* Disable the removal of unused imported identifiers from the JavaScript output. */

/* Interop Constraints */
// "isolatedModules": true, /* Ensure that each file can be safely transpiled without relying on other imports. */
Expand Down
1 change: 1 addition & 0 deletions ...eference/tsConfig/Initialized TSConfig with incorrect compiler option value/tsconfig.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -64,6 +64,7 @@
// "noEmitOnError": true, /* Disable emitting files if any type checking errors are reported. */
// "preserveConstEnums": true, /* Disable erasing `const enum` declarations in generated code. */
// "declarationDir": "./", /* Specify the output directory for generated declaration files. */
// "noErasingImportedNames": true, /* Disable the removal of unused imported identifiers from the JavaScript output. */

/* Interop Constraints */
// "isolatedModules": true, /* Ensure that each file can be safely transpiled without relying on other imports. */
Expand Down
1 change: 1 addition & 0 deletions ...ines/reference/tsConfig/Initialized TSConfig with incorrect compiler option/tsconfig.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -64,6 +64,7 @@
// "noEmitOnError": true, /* Disable emitting files if any type checking errors are reported. */
// "preserveConstEnums": true, /* Disable erasing `const enum` declarations in generated code. */
// "declarationDir": "./", /* Specify the output directory for generated declaration files. */
// "noErasingImportedNames": true, /* Disable the removal of unused imported identifiers from the JavaScript output. */

/* Interop Constraints */
// "isolatedModules": true, /* Ensure that each file can be safely transpiled without relying on other imports. */
Expand Down
1 change: 1 addition & 0 deletions ...ce/tsConfig/Initialized TSConfig with list compiler options with enum value/tsconfig.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -64,6 +64,7 @@
// "noEmitOnError": true, /* Disable emitting files if any type checking errors are reported. */
// "preserveConstEnums": true, /* Disable erasing `const enum` declarations in generated code. */
// "declarationDir": "./", /* Specify the output directory for generated declaration files. */
// "noErasingImportedNames": true, /* Disable the removal of unused imported identifiers from the JavaScript output. */

/* Interop Constraints */
// "isolatedModules": true, /* Ensure that each file can be safely transpiled without relying on other imports. */
Expand Down
1 change: 1 addition & 0 deletions ...aselines/reference/tsConfig/Initialized TSConfig with list compiler options/tsconfig.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -64,6 +64,7 @@
// "noEmitOnError": true, /* Disable emitting files if any type checking errors are reported. */
// "preserveConstEnums": true, /* Disable erasing `const enum` declarations in generated code. */
// "declarationDir": "./", /* Specify the output directory for generated declaration files. */
// "noErasingImportedNames": true, /* Disable the removal of unused imported identifiers from the JavaScript output. */

/* Interop Constraints */
// "isolatedModules": true, /* Ensure that each file can be safely transpiled without relying on other imports. */
Expand Down
1 change: 1 addition & 0 deletions ...s/should-not-trigger-recompilation-because-of-program-emit/declarationDir-is-specified.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -85,6 +85,7 @@ interface Array<T> { length: number; [n: number]: T; }
// "noEmitOnError": true, /* Disable emitting files if any type checking errors are reported. */
// "preserveConstEnums": true, /* Disable erasing `const enum` declarations in generated code. */
"declarationDir": "decls", /* Specify the output directory for generated declaration files. */
// "noErasingImportedNames": true, /* Disable the removal of unused imported identifiers from the JavaScript output. */

/* Interop Constraints */
// "isolatedModules": true, /* Ensure that each file can be safely transpiled without relying on other imports. */
Expand Down
1 change: 1 addition & 0 deletions ...gger-recompilation-because-of-program-emit/when-outDir-and-declarationDir-is-specified.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -85,6 +85,7 @@ interface Array<T> { length: number; [n: number]: T; }
// "noEmitOnError": true, /* Disable emitting files if any type checking errors are reported. */
// "preserveConstEnums": true, /* Disable erasing `const enum` declarations in generated code. */
"declarationDir": "decls", /* Specify the output directory for generated declaration files. */
// "noErasingImportedNames": true, /* Disable the removal of unused imported identifiers from the JavaScript output. */

/* Interop Constraints */
// "isolatedModules": true, /* Ensure that each file can be safely transpiled without relying on other imports. */
Expand Down
1 change: 1 addition & 0 deletions ...ates/should-not-trigger-recompilation-because-of-program-emit/when-outDir-is-specified.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -85,6 +85,7 @@ interface Array<T> { length: number; [n: number]: T; }
// "noEmitOnError": true, /* Disable emitting files if any type checking errors are reported. */
// "preserveConstEnums": true, /* Disable erasing `const enum` declarations in generated code. */
// "declarationDir": "./", /* Specify the output directory for generated declaration files. */
// "noErasingImportedNames": true, /* Disable the removal of unused imported identifiers from the JavaScript output. */

/* Interop Constraints */
// "isolatedModules": true, /* Ensure that each file can be safely transpiled without relying on other imports. */
Expand Down
1 change: 1 addition & 0 deletions ...h/programUpdates/should-not-trigger-recompilation-because-of-program-emit/with-outFile.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -85,6 +85,7 @@ interface Array<T> { length: number; [n: number]: T; }
// "noEmitOnError": true, /* Disable emitting files if any type checking errors are reported. */
// "preserveConstEnums": true, /* Disable erasing `const enum` declarations in generated code. */
// "declarationDir": "./", /* Specify the output directory for generated declaration files. */
// "noErasingImportedNames": true, /* Disable the removal of unused imported identifiers from the JavaScript output. */

/* Interop Constraints */
// "isolatedModules": true, /* Ensure that each file can be safely transpiled without relying on other imports. */
Expand Down
1 change: 1 addition & 0 deletions ...ecause-of-program-emit/without-outDir-or-outFile-is-specified-with-declaration-enabled.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -85,6 +85,7 @@ interface Array<T> { length: number; [n: number]: T; }
// "noEmitOnError": true, /* Disable emitting files if any type checking errors are reported. */
// "preserveConstEnums": true, /* Disable erasing `const enum` declarations in generated code. */
// "declarationDir": "./", /* Specify the output directory for generated declaration files. */
// "noErasingImportedNames": true, /* Disable the removal of unused imported identifiers from the JavaScript output. */

/* Interop Constraints */
// "isolatedModules": true, /* Ensure that each file can be safely transpiled without relying on other imports. */
Expand Down
1 change: 1 addition & 0 deletions ...t-trigger-recompilation-because-of-program-emit/without-outDir-or-outFile-is-specified.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -85,6 +85,7 @@ interface Array<T> { length: number; [n: number]: T; }
// "noEmitOnError": true, /* Disable emitting files if any type checking errors are reported. */
// "preserveConstEnums": true, /* Disable erasing `const enum` declarations in generated code. */
// "declarationDir": "./", /* Specify the output directory for generated declaration files. */
// "noErasingImportedNames": true, /* Disable the removal of unused imported identifiers from the JavaScript output. */

/* Interop Constraints */
// "isolatedModules": true, /* Ensure that each file can be safely transpiled without relying on other imports. */
Expand Down