Skip to content

[Master] Do not emit ESModule marker if file is not external module#14493

Merged
yuit merged 2 commits into
masterfrom
master-fixEmitExtraESModuleMark
Mar 6, 2017
Merged

[Master] Do not emit ESModule marker if file is not external module#14493
yuit merged 2 commits into
masterfrom
master-fixEmitExtraESModuleMark

Conversation

@yuit
Copy link
Copy Markdown
Contributor

@yuit yuit commented Mar 6, 2017

No description provided.

@yuit yuit requested a review from mhegazy March 6, 2017 21:38
@yuit yuit merged commit 0425175 into master Mar 6, 2017
@yuit yuit deleted the master-fixEmitExtraESModuleMark branch March 6, 2017 22:09
@Jessidhia
Copy link
Copy Markdown

What is an "external module"; a file that has import / export?

@mhegazy
Copy link
Copy Markdown
Contributor

mhegazy commented Mar 7, 2017

What is an "external module"; a file that has import / export?

correct.

@brian428
Copy link
Copy Markdown

brian428 commented Mar 8, 2017

Just wondering if there's an ETA or target version for this fix?

@mhegazy
Copy link
Copy Markdown
Contributor

mhegazy commented Mar 9, 2017

this should be already in typescript@next. currently it is planned to be in TS 2.3

namoscato added a commit to namoscato/amoscato-client that referenced this pull request Apr 1, 2017
@microsoft microsoft locked and limited conversation to collaborators Jun 19, 2018
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.

5 participants