Skip to content

feat(server): Definitions for sources compiled with declarationMap #1601

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

Merged
merged 2 commits into from
Jan 26, 2022

Conversation

atscott
Copy link
Collaborator

@atscott atscott commented Jan 20, 2022

…go to original source

The TypeScript language service has the ability to map definitions back
to their original source when the library code was compiled with
declarationMap. However, this functionality is not exposed as public
API. As a result, these methods could change in the future without our knowledge.

resolves #1588

…go to original source

The TypeScript language service has the ability to map definitions back
to their original source when the library code was compiled with
`declarationMap`. However, this functionality is not exposed as public
API. As a result, these methods could change in the future without our knowledge.

resolves angular#1588
@atscott atscott added the target: minor This PR is targeted for the next minor release label Jan 20, 2022
Copy link
Contributor

@ivanwonder ivanwonder left a comment

Choose a reason for hiding this comment

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

LGTM. Should we add a test for this?

@atscott
Copy link
Collaborator Author

atscott commented Jan 26, 2022

LGTM. Should we add a test for this?

I'm alright leaving this untested since it's a relatively niche feature and rather complicated to get the project configuration for the test correct. If you wanted to add a follow-up to test it though, I'd be happy to review.

Edit: 😅 I felt bad not having a test. The test project wasn't set up with the CLI so it was an exercise in understanding how to do things manually but I was able to get it working.

@atscott atscott force-pushed the mappedDefinitionInfo branch from e46aabf to 0ce24af Compare January 26, 2022 20:38
@atscott atscott merged commit a291516 into angular:master Jan 26, 2022
@angular-automatic-lock-bot
Copy link

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Feb 26, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
action: merge Ready to merge cla: yes target: minor This PR is targeted for the next minor release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Navigating to definition of library code compiled with declarationMap still goes to .d.ts file rather than original source
2 participants