Skip to content

refactor(language-service): stop tracking lastKnownProgram in CompilerFactory#41517

Closed
kyliau wants to merge 1 commit intoangular:masterfrom
kyliau:compiler-factory
Closed

refactor(language-service): stop tracking lastKnownProgram in CompilerFactory#41517
kyliau wants to merge 1 commit intoangular:masterfrom
kyliau:compiler-factory

Conversation

@kyliau
Copy link
Copy Markdown
Contributor

@kyliau kyliau commented Apr 8, 2021

With the work done in #41291, the compiler always tracks the last known
program, so there's no need to track the program in the compiler factory
anymore.

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.io 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

…rFactory

With the work done in angular#41291, the compiler always tracks the last known
program, so there's no need to track the program in the compiler factory
anymore.
@google-cla google-cla Bot added the cla: yes label Apr 8, 2021
@kyliau kyliau added area: language-service Issues related to Angular's VS Code language service and removed cla: yes labels Apr 8, 2021
@ngbot ngbot Bot modified the milestone: Backlog Apr 8, 2021
@google-cla google-cla Bot added the cla: yes label Apr 8, 2021
@kyliau kyliau added target: major This PR is targeted for the next major release and removed cla: yes labels Apr 8, 2021
@google-cla google-cla Bot added the cla: yes label Apr 8, 2021
@kyliau kyliau requested a review from atscott April 8, 2021 20:39
@kyliau kyliau marked this pull request as ready for review April 8, 2021 20:39
@kyliau kyliau added the action: merge The PR is ready for merge by the caretaker label Apr 8, 2021
@zarend zarend closed this in 25e46c1 Apr 9, 2021
@angular-automatic-lock-bot
Copy link
Copy Markdown

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, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

action: merge The PR is ready for merge by the caretaker area: language-service Issues related to Angular's VS Code language service cla: yes target: major This PR is targeted for the next major release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants