-
-
Notifications
You must be signed in to change notification settings - Fork 2.8k
feat: allow typescript@5.3.0-beta as devDependency #7821
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
Changes from all commits
e198efb
1e2bf7b
a13decf
b85a0eb
0f31fc2
1213b98
91f802c
d487f52
ac4bedd
281ce30
a2bef1d
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 |
---|---|---|
@@ -0,0 +1,46 @@ | ||
diff --git a/lib/analyzer/ExportAnalyzer.js b/lib/analyzer/ExportAnalyzer.js | ||
index 51f31b4f2f253d82adc9594faa58d8dc031957e0..1fddac54b08665f5a5a717fef740613ae921cb7a 100644 | ||
--- a/lib/analyzer/ExportAnalyzer.js | ||
+++ b/lib/analyzer/ExportAnalyzer.js | ||
@@ -193,7 +193,7 @@ class ExportAnalyzer { | ||
const mode = specifier && ts.isStringLiteralLike(specifier) | ||
? TypeScriptInternals_1.TypeScriptInternals.getModeForUsageLocation(importOrExportDeclaration.getSourceFile(), specifier) | ||
: undefined; | ||
- const resolvedModule = TypeScriptInternals_1.TypeScriptInternals.getResolvedModule(importOrExportDeclaration.getSourceFile(), moduleSpecifier, mode); | ||
+ const resolvedModule = TypeScriptInternals_1.TypeScriptInternals.getResolvedModule(this._program, importOrExportDeclaration.getSourceFile(), moduleSpecifier, mode); | ||
if (resolvedModule === undefined) { | ||
// The TS compiler API `getResolvedModule` cannot resolve ambient modules. Thus, to match API Extractor's | ||
// previous behavior, simply treat all ambient modules as external. This bug is tracked by | ||
@@ -666,7 +666,7 @@ class ExportAnalyzer { | ||
ts.isStringLiteralLike(importOrExportDeclaration.moduleSpecifier) | ||
? TypeScriptInternals_1.TypeScriptInternals.getModeForUsageLocation(importOrExportDeclaration.getSourceFile(), importOrExportDeclaration.moduleSpecifier) | ||
: undefined; | ||
- const resolvedModule = TypeScriptInternals_1.TypeScriptInternals.getResolvedModule(importOrExportDeclaration.getSourceFile(), moduleSpecifier, mode); | ||
+ const resolvedModule = TypeScriptInternals_1.TypeScriptInternals.getResolvedModule(this._program, importOrExportDeclaration.getSourceFile(), moduleSpecifier, mode); | ||
if (resolvedModule === undefined) { | ||
// Encountered in https://github.com/microsoft/rushstack/issues/1914. | ||
// | ||
diff --git a/lib/analyzer/TypeScriptInternals.js b/lib/analyzer/TypeScriptInternals.js | ||
index ff13db7709ccf502fe202337869d4024431636c9..217c36b0303c3bf9871edabcd48fbf6e09f18561 100644 | ||
--- a/lib/analyzer/TypeScriptInternals.js | ||
+++ b/lib/analyzer/TypeScriptInternals.js | ||
@@ -84,10 +84,16 @@ class TypeScriptInternals { | ||
* Retrieves the (cached) module resolution information for a module name that was exported from a SourceFile. | ||
* The compiler populates this cache as part of analyzing the source file. | ||
*/ | ||
- static getResolvedModule(sourceFile, moduleNameText, mode) { | ||
+ static getResolvedModule(program, sourceFile, moduleNameText, mode) { | ||
// Compiler internal: | ||
- // https://github.com/microsoft/TypeScript/blob/v4.7.2/src/compiler/utilities.ts#L161 | ||
- return ts.getResolvedModule(sourceFile, moduleNameText, mode); | ||
+ // https://github.com/microsoft/TypeScript/blob/v5.3-beta/src/compiler/types.ts#L4678 | ||
+ const result = program.getResolvedModule( | ||
+ sourceFile, | ||
+ moduleNameText, | ||
+ mode | ||
+ ); | ||
+ return result?.resolvedModule; | ||
+ | ||
} | ||
/** | ||
* Gets the mode required for module resolution required with the addition of Node16/nodenext |
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
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.
The future is now old man dot jpg
We can switch this to the 5.3 RC now
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.
yarnpkg/berry#4441 keeps hitting me shakes fist
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.
microsoft/TypeScript#56371 never mind 🥲