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(ivy): do not use MSG_ prefix for i18nPostprocess invocation (FW-779) #27468
Conversation
You can preview a58ac85 at https://pr27468-a58ac85.ngbuilds.io/. |
You can preview d52d0ff at https://pr27468-d52d0ff.ngbuilds.io/. |
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.
Please update the commit message to include the change to use unique MSG_
variable names as well (with the $$
suffix).
Closure Compiler doesn't allow non-goo.getMsg const names to start with `MSG_`, so we should use different prefix for const that references a result of the `i18nPostprocess` fn invocation. With this update we also append file-based prefix to i18n constants (via $$ postfix) to ensure the names are unique across codebase of a project (otherwise it might lead to errors while compiling a project with Closure Compiler).
d52d0ff
to
af9d031
Compare
You can preview af9d031 at https://pr27468-af9d031.ngbuilds.io/. |
…79) (angular#27468) Closure Compiler doesn't allow non-goo.getMsg const names to start with `MSG_`, so we should use different prefix for const that references a result of the `i18nPostprocess` fn invocation. With this update we also append file-based prefix to i18n constants (via $$ postfix) to ensure the names are unique across codebase of a project (otherwise it might lead to errors while compiling a project with Closure Compiler). PR Close angular#27468
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. |
Closure Compiler doesn't allow non-goo.getMsg const names to start with
MSG_
, so we should use different prefix for const that references a result of thei18nPostprocess
fn invocation.Also, as a part of this PR, const names received file based prefix to avoid naming collisions that Closure Compiler doesn't tolerate (there should be unique instance of
goog.getMsg
call across the whole app).PR Type
What kind of change does this PR introduce?
Does this PR introduce a breaking change?