Allow declarationMap to be emitted when transpiling declarations and option is enabled by sheetalkamat · Pull Request #59337 · microsoft/TypeScript

Skip to content

Navigation Menu

Sign in

Appearance settings

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up

Appearance settings

Conversation

@sheetalkamat

Copy link Copy Markdown

Member

Fixes #59313

MichaelMitchell-at reacted with heart emoji

@codecov-commenter

Copy link Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Please upload report for BASE (main@e7e8135). Learn more about missing BASE report.

Additional details and impacted files
@@           Coverage Diff           @@
##             main   #59337   +/-   ##
=======================================
  Coverage        ?   94.07%           
=======================================
  Files           ?      587           
  Lines           ?   300852           
  Branches        ?     5082           
=======================================
  Hits            ?   283023           
  Misses          ?    12747           
  Partials        ?     5082           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@DanielRosenwasser

Copy link Copy Markdown

Member

Should this be picked back to 5.5?

@jakebailey

Copy link Copy Markdown

Member

Probably?

@sheetalkamat sheetalkamat merged commit afa03f0 into main

Jul 18, 2024

@sheetalkamat sheetalkamat deleted the transpileDeclarations branch

July 18, 2024 00:07

@sheetalkamat sheetalkamat added this to the TypeScript 5.5.4 milestone

Jul 18, 2024

@sheetalkamat

Copy link Copy Markdown

Member Author

@typescript-bot cherry-pick this to release-5.5

typescript-bot reacted with thumbs up emoji

@typescript-bot

Copy link Copy Markdown

Collaborator

typescript-bot commented

Jul 18, 2024

edited

Loading

Starting jobs; this comment will be updated as builds start and complete.

Command Status Results
cherry-pick this to release-5.5 ✅ Started ✅ Results

@typescript-bot

Copy link Copy Markdown

Collaborator

Hey, @sheetalkamat! I've created #59344 for you.

DanielRosenwasser pushed a commit that referenced this pull request

Jul 19, 2024
…e-5.5 (#59344)

Co-authored-by: Sheetal Nandi <shkamat@microsoft.com>

@sandersn sandersn removed this from PR Backlog

Apr 22, 2025

@microsoft microsoft locked as resolved and limited conversation to collaborators

Oct 16, 2025

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@weswigham weswigham weswigham approved these changes

@jakebailey jakebailey Awaiting requested review from jakebailey

@DanielRosenwasser DanielRosenwasser Awaiting requested review from DanielRosenwasser

Labels

Author: Team For Uncommitted Bug

PR for untriaged, rejected, closed or missing bug

Projects

None yet

Milestone

TypeScript 5.5.4

Development

Successfully merging this pull request may close these issues.

[transpileDeclaration API] ts.transpileDeclaration always returns empty sourceMapText

7 participants

@sheetalkamat @codecov-commenter @DanielRosenwasser @jakebailey @typescript-bot @weswigham @sandersn