Skip to content

refactor(core): move directive matching logic to a separate file#59843

Closed
pkozlowski-opensource wants to merge 4 commits intoangular:mainfrom
pkozlowski-opensource:runtime_refactor_element_first_pass
Closed

refactor(core): move directive matching logic to a separate file#59843
pkozlowski-opensource wants to merge 4 commits intoangular:mainfrom
pkozlowski-opensource:runtime_refactor_element_first_pass

Conversation

@pkozlowski-opensource
Copy link
Copy Markdown
Member

Move directive matching logic to a separate file.

@angular-robot angular-robot bot added the area: core Issues related to the framework runtime label Feb 3, 2025
@ngbot ngbot bot added this to the Backlog milestone Feb 3, 2025
@pkozlowski-opensource pkozlowski-opensource force-pushed the runtime_refactor_element_first_pass branch from 77309cd to 1174e64 Compare February 3, 2025 23:12
Move directive matching logic to a separate file.
@pkozlowski-opensource pkozlowski-opensource force-pushed the runtime_refactor_element_first_pass branch from 1174e64 to 07d12ff Compare February 4, 2025 03:31
Reuse element first create pass in ComponentRef.
Reuse element end first creation pass in ComponentRef impl.
@pkozlowski-opensource pkozlowski-opensource force-pushed the runtime_refactor_element_first_pass branch from badfaae to 44668cc Compare February 4, 2025 15:43
@pkozlowski-opensource pkozlowski-opensource marked this pull request as ready for review February 4, 2025 15:44
@pkozlowski-opensource pkozlowski-opensource added action: review The PR is still awaiting reviews from at least one requested reviewer target: patch This PR is targeted for the next patch release labels Feb 4, 2025
Some minor code cleanups after code changes in
previous commits.
@pkozlowski-opensource pkozlowski-opensource removed the action: review The PR is still awaiting reviews from at least one requested reviewer label Feb 5, 2025
@pkozlowski-opensource pkozlowski-opensource added the action: merge The PR is ready for merge by the caretaker label Feb 5, 2025
@alxhub
Copy link
Copy Markdown
Member

alxhub commented Feb 5, 2025

This PR was merged into the repository by commit 41b1a7c.

The changes were merged into the following branches: main, 19.1.x

@alxhub alxhub closed this in e3dcf52 Feb 5, 2025
alxhub pushed a commit that referenced this pull request Feb 5, 2025
Reuse element first create pass in ComponentRef.

PR Close #59843
alxhub pushed a commit that referenced this pull request Feb 5, 2025
Reuse element end first creation pass in ComponentRef impl.

PR Close #59843
alxhub pushed a commit that referenced this pull request Feb 5, 2025
Some minor code cleanups after code changes in
previous commits.

PR Close #59843
alxhub pushed a commit that referenced this pull request Feb 5, 2025
)

Move directive matching logic to a separate file.

PR Close #59843
alxhub pushed a commit that referenced this pull request Feb 5, 2025
Reuse element first create pass in ComponentRef.

PR Close #59843
alxhub pushed a commit that referenced this pull request Feb 5, 2025
Reuse element end first creation pass in ComponentRef impl.

PR Close #59843
alxhub pushed a commit that referenced this pull request Feb 5, 2025
Some minor code cleanups after code changes in
previous commits.

PR Close #59843
PrajaktaB27 pushed a commit to PrajaktaB27/angular that referenced this pull request Feb 7, 2025
PrajaktaB27 pushed a commit to PrajaktaB27/angular that referenced this pull request Feb 7, 2025
PrajaktaB27 pushed a commit to PrajaktaB27/angular that referenced this pull request Feb 7, 2025
Reuse element end first creation pass in ComponentRef impl.

PR Close angular#59843
PrajaktaB27 pushed a commit to PrajaktaB27/angular that referenced this pull request Feb 7, 2025
Some minor code cleanups after code changes in
previous commits.

PR Close angular#59843
@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 Mar 8, 2025
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: core Issues related to the framework runtime target: patch This PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants