Make GetAttribute use reflection

[paddycarver]

Module version

v0.2.0

Use-cases

See #46.

Attempted Solutions

#46

Proposal

Rather than having GetAttribute return an attr.Value, let's mirror Get and have it take a target interface{} that it reflects into. Otherwise, people are just going to need to use tfsdk.ValueOf or type-assert out of it anyways, so there's no real point in surfacing the attr.Value version. If people want the attr.Value, they can make target an attr.Value implementation, so I don't know that we lose anything here.

This lets us mirror #133 for reflect.Into and mirrors how SetAttribute works, so it just feels more consistent.

References

• tfsdk: Properly set attribute path in (Plan).SetAttribute() and (State).SetAttribute() reflection diagnostics #133
• Implement State, State.Get, State.GetAttribute, Config, Plan #46

[github-actions]

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.
If you have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.