Thanks to visit codestin.com
Credit goes to github.com

Skip to content

feat(language-service): Support importing the external module's expor… #61122

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

Closed
wants to merge 1 commit into from

Conversation

ivanwonder
Copy link
Contributor

@ivanwonder ivanwonder commented May 5, 2025

…t about the angular metadata.

This works for the code actions and completion. When the ls wants to complete all importable angular metadata in the template, the ls will invoke the function ls.getCompletionsAtPosition and filter the item about the angular. When the developer selects an item, the ls will get the module specifier from the code action return by the ls.getCompletionEntryDetails.

PR Checklist

Please check if your PR fulfills the following requirements:

PR Type

What kind of change does this PR introduce?

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • CI related changes
  • Documentation content changes
  • angular.dev application / infrastructure changes
  • Other... Please describe:

What is the current behavior?

Issue Number: N/A

What is the new behavior?

Does this PR introduce a breaking change?

  • Yes
  • No

Other information

@angular-robot angular-robot bot added detected: feature PR contains a feature commit area: language-service Issues related to Angular's VS Code language service labels May 5, 2025
@ngbot ngbot bot added this to the Backlog milestone May 5, 2025
@ivanwonder ivanwonder marked this pull request as ready for review May 5, 2025 12:39
@pullapprove pullapprove bot requested a review from kirjs May 5, 2025 12:39
@kirjs kirjs removed their request for review May 6, 2025 16:11
@pullapprove pullapprove bot requested a review from kirjs May 6, 2025 16:11
@clydin clydin self-requested a review May 6, 2025 16:13
@kirjs kirjs removed their request for review May 6, 2025 18:39
@pullapprove pullapprove bot requested a review from kirjs May 6, 2025 18:39
@kirjs kirjs removed their request for review May 22, 2025 17:52
@pullapprove pullapprove bot requested a review from kirjs May 22, 2025 17:52
@clydin clydin requested review from atscott and removed request for kirjs May 22, 2025 17:57
Copy link
Contributor

@atscott atscott left a comment

Choose a reason for hiding this comment

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

Thanks for looking into this. Leveraging the TS language service for this is a cool idea.

@ivanwonder ivanwonder requested a review from atscott May 23, 2025 08:04
@ivanwonder ivanwonder force-pushed the ts-global-completions branch from acd5552 to 9ba7143 Compare May 23, 2025 12:55
@ivanwonder ivanwonder requested a review from atscott May 23, 2025 15:17
@ivanwonder ivanwonder force-pushed the ts-global-completions branch from 352b13f to 606b4da Compare May 28, 2025 01:45
…t about the angular metadata.

This works for the code actions and completion. When the ls wants to complete
all importable angular metadata in the template, the ls will invoke the function
`ls.getCompletionsAtPosition` and filter the item about the angular. When the
developer selects an item, the ls will get the module specifier from the code action
return by the `ls.getCompletionEntryDetails`.
@ivanwonder ivanwonder force-pushed the ts-global-completions branch from 606b4da to 4607783 Compare May 29, 2025 02:10
@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 labels Jun 4, 2025
@thePunderWoman thePunderWoman removed the request for review from clydin June 4, 2025 17:21
@kirjs
Copy link
Contributor

kirjs commented Jun 4, 2025

This PR was merged into the repository by commit cf55d1b.

The changes were merged into the following branches: main

@kirjs kirjs closed this in cf55d1b Jun 4, 2025
@ivanwonder ivanwonder deleted the ts-global-completions branch June 5, 2025 01:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
action: merge The PR is ready for merge by the caretaker area: language-service Issues related to Angular's VS Code language service detected: feature PR contains a feature commit target: minor This PR is targeted for the next minor release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants