Skip to content

Instead of emitting js as well as d.ts files when only d.ts emit could change, emit only d.ts files since program.emit has that option#33358

Merged
sheetalkamat merged 9 commits into
masterfrom
dtsOnlyEmit
Sep 11, 2019
Merged

Instead of emitting js as well as d.ts files when only d.ts emit could change, emit only d.ts files since program.emit has that option#33358
sheetalkamat merged 9 commits into
masterfrom
dtsOnlyEmit

Conversation

@sheetalkamat
Copy link
Copy Markdown
Member

No description provided.

Comment thread src/harness/vfs.ts Outdated
Comment thread src/harness/vfs.ts
Comment thread src/compiler/types.ts Outdated
//// [/src/first/bin/first-output.d.ts] file written with same contents
//// [/src/first/bin/first-output.d.ts.map] file written with same contents
//// [/src/first/bin/first-output.d.ts.map.baseline.txt] file written with same contents
//// [/src/first/bin/first-output.js]
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is cool 👍

Nice work.

@sheetalkamat sheetalkamat merged commit 6995a90 into master Sep 11, 2019
@sheetalkamat sheetalkamat deleted the dtsOnlyEmit branch September 11, 2019 20:11
@microsoft microsoft locked as resolved and limited conversation to collaborators Oct 21, 2025
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.

2 participants