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
[schema] Emit a warning when using id or urn as resource properties #15060
Conversation
Changelog[uncommitted] (2024-01-17)Miscellaneous
|
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.
Is there an easy way to test the warnings?
LGTM otherwise.
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.
Yeh this needs to re-add the tests removed in #15025.
Unfortunately because ImportSpec
only returns errors not diagnostics you'll need to call bindSpec
in the test instead.
Also a new test to check that for a component resource |
6573f85
to
79d96f0
Compare
a4afd2a
to
b2c8e9e
Compare
### Features - [sdk/go] Add WithValue to pulumi.Context [#15149](#15149) ### Bug Fixes - [cli] Fix a panic when user's home directory could not be looked up. [#15160](#15160) - [cli/display] Fix JSON/YAML value diff displays. - [sdk/go] Update github.com/cloudflare/circl to v1.3.7. [#15151](#15151) - [cli/import] --import-file now omits empty resource lists. [#15132](#15132) - [sdk/python] Allow remote components to use output property called id ### Miscellaneous - [sdkgen] Emit a warning when using id or urn as resource outputs [#15060](#15060)
### Features - [sdk/go] Add WithValue to pulumi.Context [#15149](#15149) ### Bug Fixes - [cli] Fix a panic when user's home directory could not be looked up. [#15160](#15160) - [cli/display] Fix JSON/YAML value diff displays. [#15171](#15171) - [sdk/go] Update github.com/cloudflare/circl to v1.3.7. [#15151](#15151) - [cli/import] --import-file now omits empty resource lists. [#15132](#15132) - [sdk/python] Allow remote components to use output property called id [#15115](#15115) ### Miscellaneous - [sdkgen] Emit a warning when using id or urn as resource outputs [#15060](#15060)
Description
Emits a warning when binding properties of resources if we encounter a property with name
urn
or when we encounter a property with nameid
(only for resources which are not components / MLCs). This is a follow up PR for #15025Checklist
make tidy
to update any new dependenciesmake lint
to verify my code passes the lint checkgofumpt
make changelog
and committed thechangelog/pending/<file>
documenting my change