Implement State, State.Get, State.GetAttribute, Config, Plan#46
Conversation
Co-authored-by: Paddy <paddy@hashicorp.com>
|
It turns out that nested attributes did not need the Please take a look at the tests to see if this provides the appropriate UX. |
paddycarver
left a comment
There was a problem hiding this comment.
Some minor feedback on this one. I think the only part that's super important is the object using the wrong attribute path step type.
|
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. |
The changes to
internal/reflectonmainmade rather a mess ofkatystate3. We're up tokatystate5now.The main work here is the addition of
ApplyTerraform5AttributePathStepto all schema types, implementing thetftypes.AttributePathStepperinterface so that we can traverse the schema withtftypes.WalkAttributePath.