Skip to content

refactor(core): move logic out of shared instructions code#59453

Closed
pkozlowski-opensource wants to merge 1 commit intoangular:mainfrom
pkozlowski-opensource:runtime_refactor_shared_queries
Closed

refactor(core): move logic out of shared instructions code#59453
pkozlowski-opensource wants to merge 1 commit intoangular:mainfrom
pkozlowski-opensource:runtime_refactor_shared_queries

Conversation

@pkozlowski-opensource
Copy link
Copy Markdown
Member

This is first of a series of refactorings that moves code around such that logic from the shared instruction file is dispatched to the relevant functional parts.

@angular-robot angular-robot bot added the area: core Issues related to the framework runtime label Jan 9, 2025
@ngbot ngbot bot added this to the Backlog milestone Jan 9, 2025
@angular-robot angular-robot bot added area: core Issues related to the framework runtime and removed area: core Issues related to the framework runtime labels Jan 9, 2025
@pkozlowski-opensource pkozlowski-opensource force-pushed the runtime_refactor_shared_queries branch from 32bf202 to 7ef67f0 Compare January 9, 2025 18:27
@angular-robot angular-robot bot added area: core Issues related to the framework runtime and removed area: core Issues related to the framework runtime labels Jan 9, 2025
@pkozlowski-opensource pkozlowski-opensource marked this pull request as ready for review January 9, 2025 18:28
@pkozlowski-opensource pkozlowski-opensource changed the title refactor(core): move query logic out of shared instructions code refactor(core): move logic out of shared instructions code Jan 9, 2025
@pkozlowski-opensource pkozlowski-opensource force-pushed the runtime_refactor_shared_queries branch from 7ef67f0 to 022a6cb Compare January 9, 2025 19:11
@angular-robot angular-robot bot added area: core Issues related to the framework runtime and removed area: core Issues related to the framework runtime labels Jan 9, 2025
@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 Jan 9, 2025
@AndrewKushnir AndrewKushnir added action: cleanup The PR is in need of cleanup, either due to needing a rebase or in response to comments from reviews and removed action: review The PR is still awaiting reviews from at least one requested reviewer labels Jan 9, 2025
@pkozlowski-opensource pkozlowski-opensource force-pushed the runtime_refactor_shared_queries branch from 022a6cb to d6efaa8 Compare January 10, 2025 09:58
@angular-robot angular-robot bot added area: core Issues related to the framework runtime and removed area: core Issues related to the framework runtime labels Jan 10, 2025
@ngbot ngbot bot modified the milestone: Backlog Jan 10, 2025
@pkozlowski-opensource pkozlowski-opensource added action: merge The PR is ready for merge by the caretaker action: cleanup The PR is in need of cleanup, either due to needing a rebase or in response to comments from reviews and removed action: cleanup The PR is in need of cleanup, either due to needing a rebase or in response to comments from reviews action: merge The PR is ready for merge by the caretaker labels Jan 10, 2025
@pkozlowski-opensource pkozlowski-opensource force-pushed the runtime_refactor_shared_queries branch from d6efaa8 to 1f87aae Compare January 14, 2025 11:28
This is first of a series of refactorings that moves code
around such that logic from the shared instruction file
is dispatched to the relevant functional parts.
@pkozlowski-opensource pkozlowski-opensource force-pushed the runtime_refactor_shared_queries branch from 1f87aae to c3da736 Compare January 15, 2025 10:34
@pkozlowski-opensource pkozlowski-opensource added action: merge The PR is ready for merge by the caretaker merge: caretaker note Alert the caretaker performing the merge to check the PR for an out of normal action needed or note and removed action: cleanup The PR is in need of cleanup, either due to needing a rebase or in response to comments from reviews labels Jan 15, 2025
@pkozlowski-opensource
Copy link
Copy Markdown
Member Author

caretaker note: I believe that the G3 failures are pre-existing and unrelated.

@AndrewKushnir AndrewKushnir added target: minor This PR is targeted for the next minor release and removed target: patch This PR is targeted for the next patch release labels Jan 15, 2025
@AndrewKushnir
Copy link
Copy Markdown
Contributor

This PR was merged into the repository by commit af99c6c.

The changes were merged into the following branches: main

@AndrewKushnir
Copy link
Copy Markdown
Contributor

@pkozlowski-opensource FYI, this PR conflicted with the patch branch (19.1.x), so I've merged it into the main only. Could you please create a patch version of this change (to minimize the diff between branches and avoid future merge conflicts) when you get a chance?

pkozlowski-opensource added a commit that referenced this pull request Jan 16, 2025
This is first of a series of refactorings that moves code
around such that logic from the shared instruction file
is dispatched to the relevant functional parts.

PR Close #59453
PrajaktaB27 pushed a commit to PrajaktaB27/angular that referenced this pull request Feb 7, 2025
…9453)

This is first of a series of refactorings that moves code
around such that logic from the shared instruction file
is dispatched to the relevant functional parts.

PR Close angular#59453
@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 Feb 15, 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 merge: caretaker note Alert the caretaker performing the merge to check the PR for an out of normal action needed or note 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