Skip to content

fix(compiler): remove container index from conditional instruction#55190

Closed
crisbeto wants to merge 1 commit intoangular:mainfrom
crisbeto:conditional-container-index
Closed

fix(compiler): remove container index from conditional instruction#55190
crisbeto wants to merge 1 commit intoangular:mainfrom
crisbeto:conditional-container-index

Conversation

@crisbeto
Copy link
Member

@crisbeto crisbeto commented Apr 3, 2024

Stops passing in the containerIndex argument to the conditional instruction since it isn't being used anymore.

@crisbeto crisbeto added 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 area: compiler Issues related to `ngc`, Angular's template compiler labels Apr 3, 2024
@ngbot ngbot bot modified the milestone: Backlog Apr 3, 2024
@crisbeto crisbeto force-pushed the conditional-container-index branch from 5f678cf to 7948c39 Compare April 3, 2024 14:27
Copy link
Member

@pkozlowski-opensource pkozlowski-opensource left a comment

Choose a reason for hiding this comment

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

LGTM

Reviewed-for: fw-core

@crisbeto crisbeto force-pushed the conditional-container-index branch 2 times, most recently from 5fc8aa2 to 0b52f47 Compare April 3, 2024 14:51
@crisbeto crisbeto modified the milestones: Backlog, v18-candidates Apr 3, 2024
Stops passing in the `containerIndex` argument to the `conditional` instruction since it isn't being used anymore.
@crisbeto crisbeto force-pushed the conditional-container-index branch from 0b52f47 to 98b7020 Compare April 15, 2024 15:00
@crisbeto crisbeto marked this pull request as ready for review April 15, 2024 15:32
@crisbeto crisbeto added action: merge The PR is ready for merge by the caretaker and removed action: review The PR is still awaiting reviews from at least one requested reviewer labels Apr 15, 2024
@pkozlowski-opensource
Copy link
Member

This PR was merged into the repository by commit 7d5bc1c.

@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 17, 2024
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: compiler Issues related to `ngc`, Angular's template compiler 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