-
Notifications
You must be signed in to change notification settings - Fork 12.5k
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
Exports + .d.ts emit #2139
Exports + .d.ts emit #2139
Conversation
…rectly or indirectly
…ts emit Emit those new declarations asynchronously since they are otherwise not visible
Conflicts: tests/baselines/reference/es6ImportDefaultBinding.errors.txt tests/baselines/reference/es6ImportDefaultBindingFollowedWithNamedImport.errors.txt tests/baselines/reference/es6ImportDefaultBindingFollowedWithNamedImportInEs5.errors.txt tests/baselines/reference/es6ImportDefaultBindingFollowedWithNamespaceBinding.errors.txt tests/baselines/reference/es6ImportDefaultBindingFollowedWithNamespaceBindingInEs5.errors.txt tests/baselines/reference/es6ImportDefaultBindingInEs5.errors.txt
Sorry to bother, do you know when this will be landing on master? It’s necessary for me to update dts-generator for use with ES6 modules. |
Hopefully by end of week. Sheetal has been sick for a while now, @vladima is picking it up. |
Great, thanks! I hope you feel better soon, @sheetalkamat ! Being sick is no fun. 😷 |
@@ -1837,6 +1861,40 @@ module ts { | |||
} | |||
} | |||
|
|||
function setDeclarationsOfIdentifierAsVisible(node: Identifier): Node[]{ |
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 we find a better name.
👍 |
This PR is an aggregation of #2008 and #2025 and subsumes them.