Skip to content

fix(migrations): resolve infinite loop for a single line element with a long tag name and angle bracket on a new line#54588

Closed
crisbeto wants to merge 1 commit intoangular:mainfrom
crisbeto:54587/control-flow-migration
Closed

fix(migrations): resolve infinite loop for a single line element with a long tag name and angle bracket on a new line#54588
crisbeto wants to merge 1 commit intoangular:mainfrom
crisbeto:54587/control-flow-migration

Conversation

@crisbeto
Copy link
Member

Fixes an edge case where a single-line elemnt with a long tag name a closing bracket on a new line was putting the control flow migration into an infinite loop.

Fixes #54587.

… a long tag name and angle bracket on a new line

Fixes an edge case where a single-line elemnt with a long tag name a closing bracket on a new line was putting the control flow migration into an infinite loop.

Fixes angular#54587.
@crisbeto crisbeto 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 23, 2024
Copy link
Contributor

@jessicajaniuk jessicajaniuk left a comment

Choose a reason for hiding this comment

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

Gotta love these edge cases. LGTM

@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 Feb 23, 2024
@jessicajaniuk jessicajaniuk added the area: migrations Issues related to `ng update`/`ng generate` migrations label Feb 23, 2024
@ngbot ngbot bot added this to the Backlog milestone Feb 23, 2024
@dylhunn
Copy link
Contributor

dylhunn commented Feb 23, 2024

This PR was merged into the repository by commit 71e0c7d.

dylhunn pushed a commit that referenced this pull request Feb 23, 2024
… a long tag name and angle bracket on a new line (#54588)

Fixes an edge case where a single-line elemnt with a long tag name a closing bracket on a new line was putting the control flow migration into an infinite loop.

Fixes #54587.

PR Close #54588
@dylhunn dylhunn closed this in 71e0c7d Feb 23, 2024
@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 Mar 25, 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: migrations Issues related to `ng update`/`ng generate` migrations target: patch This PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

control-flow command freeze

3 participants