FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Fix assert with declaration emit for export specifier for a global by mhegazy · Pull Request #3641 · microsoft/TypeScript · GitHub

Fix assert with declaration emit for export specifier for a global - #3641

Merged
Mohamed Hegazy (mhegazy) merged 1 commit into
masterfrom
exportSepcifierInDeclarations
Jun 30, 2015
Merged

Fix assert with declaration emit for export specifier for a global#3641
Mohamed Hegazy (mhegazy) merged 1 commit into
masterfrom
exportSepcifierInDeclarations

Conversation

Copy link
Copy Markdown
Contributor

Fixes #3640

Copy link
Copy Markdown
Contributor

This fixes the assertion error that I have seen. I'm getting a bunch of new errors on Angular, but I think those are because of the more precise type checking, now that our Promise doesn't end up being effectively any anymore.

So cannot confirm 100% success yet, but this looks good.

Copy link
Copy Markdown
Contributor

👍

Mohamed Hegazy (mhegazy) added a commit that referenced this pull request Jun 30, 2015
Fix assert with declaration emit for export specifier for a global
Mohamed Hegazy (mhegazy) merged commit fe46122 into master Jun 30, 2015
Mohamed Hegazy (mhegazy) deleted the exportSepcifierInDeclarations branch June 30, 2015 00:28
Wesley Wigham (weswigham) added a commit that referenced this pull request Nov 9, 2017
* Badness

* Revert #3641, whose original bug has been fixed by other means

* Add another repro
Microsoft (microsoft) locked and limited conversation to collaborators Jun 19, 2018
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants


Back | FazBrowse Home | New Git URL