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): avoid conflicts with some greek letters in control flow migration #55113
fix(migrations): avoid conflicts with some greek letters in control flow migration #55113
Conversation
…low migration The control flow migration was using a couple of Greek letters as placeholders. This ended up conflicting with templates authored in Greek. These changes use a more obscure placeholder to make conflicts less likely. It also moves the placeholder generation to a centralized function so it's easier to make changes if we decide to update the pattern again. Fixes angular#55085.
export function getPlaceholder( | ||
value: string, kind: PlaceholderKind = PlaceholderKind.Default): string { | ||
const name = `<<<ɵɵngControlFlowMigration_${kind}ɵɵ>>>`; | ||
return `___${name}${value}${name}___`; |
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.
Technically things can still break if somebody is wrapping their content in ___<<<ɵɵngControlFlowMigration_0ɵɵ>>><<<ɵɵngControlFlowMigration_0ɵɵ>>>___
but hopefully it's a bit less likely than before.
Caretaker note: the presubmit is failing because of one unrelated flaky target. |
Nice, thanks for the fast response to the issue! I now wish had reported it sooner 😊 |
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.
LGTM! Thanks for fixing this, @crisbeto
…low migration (#55113) The control flow migration was using a couple of Greek letters as placeholders. This ended up conflicting with templates authored in Greek. These changes use a more obscure placeholder to make conflicts less likely. It also moves the placeholder generation to a centralized function so it's easier to make changes if we decide to update the pattern again. Fixes #55085. PR Close #55113
This PR was merged into the repository by commit 0c20c40. |
…low migration (angular#55113) The control flow migration was using a couple of Greek letters as placeholders. This ended up conflicting with templates authored in Greek. These changes use a more obscure placeholder to make conflicts less likely. It also moves the placeholder generation to a centralized function so it's easier to make changes if we decide to update the pattern again. Fixes angular#55085. PR Close angular#55113
…low migration (angular#55113) The control flow migration was using a couple of Greek letters as placeholders. This ended up conflicting with templates authored in Greek. These changes use a more obscure placeholder to make conflicts less likely. It also moves the placeholder generation to a centralized function so it's easier to make changes if we decide to update the pattern again. Fixes angular#55085. PR Close angular#55113
…low migration (angular#55113) The control flow migration was using a couple of Greek letters as placeholders. This ended up conflicting with templates authored in Greek. These changes use a more obscure placeholder to make conflicts less likely. It also moves the placeholder generation to a centralized function so it's easier to make changes if we decide to update the pattern again. Fixes angular#55085. PR Close angular#55113
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. |
The control flow migration was using a couple of Greek letters as placeholders. This ended up conflicting with templates authored in Greek.
These changes use a more obscure placeholder to make conflicts less likely. It also moves the placeholder generation to a centralized function so it's easier to make changes if we decide to update the pattern again.
Fixes #55085.