New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix(migrations): Add support for nested structures inside a switch statement #52358
Conversation
…atement This updates the code to handle switches more elegantly in line with how the other blocks are handled. This allows nesting to be handled just like other blocks.
bbf7973
to
394b023
Compare
Caretaker note: This is safe to merge. The G3 failures are unrelated. |
This PR was merged into the repository by commit 9692aeb. |
@@ -134,10 +134,12 @@ export function migrateTemplate(template: string): {migrated: string|null, error | |||
|
|||
// start from top of template | |||
// loop through each element | |||
visitor.elements[0].hasLineBreaks = hasLineBreaks; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This crashes the migration if there are files without any control flow statements
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the heads up. #52399 should fix this.
…atement (angular#52358) This updates the code to handle switches more elegantly in line with how the other blocks are handled. This allows nesting to be handled just like other blocks. PR Close angular#52358
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
…atement (angular#52358) This updates the code to handle switches more elegantly in line with how the other blocks are handled. This allows nesting to be handled just like other blocks. PR Close angular#52358
This updates the code to handle switches more elegantly in line with how the other blocks are handled. This allows nesting to be handled just like other blocks.
PR Checklist
Please check if your PR fulfills the following requirements:
PR Type
What kind of change does this PR introduce?
Does this PR introduce a breaking change?