Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix(language-service): implement getDefinitionAtPosition for Angular templates #55269

Closed
wants to merge 1 commit into from

Conversation

clydin
Copy link
Member

@clydin clydin commented Apr 9, 2024

The getDefinitionAtPosition function may be called by consumers instead of getDefinitionAndBoundSpan with the later of which already implemented. The getDefinitionAtPosition result is a subset of what getDefinitionAndBoundSpan returns and currently delegates to that function.

…templates

The `getDefinitionAtPosition` function may be called by consumers instead
of `getDefinitionAndBoundSpan` with the later of which already implemented.
The `getDefinitionAtPosition` result is a subset of what `getDefinitionAndBoundSpan`
returns and currently delegates to that function.
@clydin clydin added action: review The PR is still awaiting reviews from at least one requested reviewer area: language-service Issues related to Angular's VS Code language service target: major This PR is targeted for the next major release labels Apr 9, 2024
@clydin clydin requested a review from atscott April 9, 2024 20:05
@ngbot ngbot bot added this to the Backlog milestone Apr 9, 2024
@atscott atscott added action: merge The PR is ready for merge by the caretaker target: minor This PR is targeted for the next minor release and removed action: review The PR is still awaiting reviews from at least one requested reviewer target: major This PR is targeted for the next major release labels Apr 9, 2024
@atscott
Copy link
Contributor

atscott commented Apr 9, 2024

This PR was merged into the repository by commit bd236cc.

@atscott atscott closed this in bd236cc Apr 9, 2024
@clydin clydin deleted the ls/definition-at-position branch April 9, 2024 21:05
iteriani pushed a commit to iteriani/angular that referenced this pull request Apr 11, 2024
…templates (angular#55269)

The `getDefinitionAtPosition` function may be called by consumers instead
of `getDefinitionAndBoundSpan` with the later of which already implemented.
The `getDefinitionAtPosition` result is a subset of what `getDefinitionAndBoundSpan`
returns and currently delegates to that function.

PR Close angular#55269
@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 May 10, 2024
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
action: merge The PR is ready for merge by the caretaker area: language-service Issues related to Angular's VS Code language service target: minor This PR is targeted for the next minor release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants