-
Notifications
You must be signed in to change notification settings - Fork 230
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
Prototype exposing cty.Values from ResourceData. #802
Conversation
71bc10b
to
6fb4a1a
Compare
92b2e1f
to
352c0d2
Compare
Supporting tests for hashicorp/terraform-plugin-sdk#802. Testing the new cty stuff hanging off the ResourceData and ResourceDiff types.
Supporting tests for hashicorp/terraform-plugin-sdk#802. Testing the new cty stuff hanging off the ResourceData and ResourceDiff types.
Supported by hashicorp/terraform-provider-corner#29. |
Supporting tests for hashicorp/terraform-plugin-sdk#802. Testing the new cty stuff hanging off the ResourceData and ResourceDiff types.
Supporting tests for hashicorp/terraform-plugin-sdk#802. Testing the new cty stuff hanging off the ResourceData and ResourceDiff types.
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 to me 🚀 Nice work threading everything through.
Supporting tests for hashicorp/terraform-plugin-sdk#802. Testing the new cty stuff hanging off the ResourceData and ResourceDiff types.
Supporting tests for hashicorp/terraform-plugin-sdk#802. Testing the new cty stuff hanging off the ResourceData and ResourceDiff types.
I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active contributions. |
Fixes #793.
We actually create a
terraform.InstanceState
when handling the gRPC methods, and then a ResourceData is created from that. So, like with ProviderMeta functionality (#405) we're following the same path and threading our cty values through the instance state.This still needs unit tests.