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
Escape deprecation messages before interpolating into source text #9371
Conversation
Diff for pulumi-random with merge commit ee0e97d |
Diff for pulumi-azuread with merge commit ee0e97d |
Diff for pulumi-kubernetes with merge commit ee0e97d |
Diff for pulumi-gcp with merge commit ee0e97d |
Diff for pulumi-azure with merge commit ee0e97d |
Diff for pulumi-aws with merge commit ee0e97d |
Diff for pulumi-random with merge commit 7fc80f3 |
Diff for pulumi-azuread with merge commit 7fc80f3 |
Diff for pulumi-kubernetes with merge commit 7fc80f3 |
Diff for pulumi-gcp with merge commit 7fc80f3 |
Diff for pulumi-azure with merge commit 7fc80f3 |
Diff for pulumi-aws with merge commit 7fc80f3 |
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 looks good. I have a question and a nit, but otherwise LGTM.
Nit: since this is a general function, I think we should put it in nodejs/utilities.go
instead of nodejs/gen.go
. Feel free to ignore this.
Are there any other strings we generate that should be escaped? Comments are /* */
style, so it's not a problem there. I don't think we escape constant values, but we probably should.
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 looks great thank you!
Diff for pulumi-azure-native with merge commit ee0e97d |
It looks like other strings are using module/type/property names which should already be string safe. |
Diff for pulumi-azuread with merge commit 1fca65f |
Diff for pulumi-random with merge commit 1fca65f |
Diff for pulumi-kubernetes with merge commit 1fca65f |
Diff for pulumi-gcp with merge commit 1fca65f |
Diff for pulumi-azure with merge commit 1fca65f |
Diff for pulumi-random with merge commit 66b13bd |
Diff for pulumi-azuread with merge commit 66b13bd |
Diff for pulumi-kubernetes with merge commit 66b13bd |
Diff for pulumi-aws with merge commit 1fca65f |
Diff for pulumi-gcp with merge commit 66b13bd |
Diff for pulumi-azure with merge commit 66b13bd |
Diff for pulumi-aws with merge commit 66b13bd |
Diff for pulumi-azure-native with merge commit 7fc80f3 |
Diff for pulumi-azure-native with merge commit 1fca65f |
Diff for pulumi-azure-native with merge commit 66b13bd |
Description
Fixes #9342
Checklist