-
Notifications
You must be signed in to change notification settings - Fork 9.4k
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
stacks: support unknown for_each on embedded stacks #35119
Conversation
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.
I have some requests around comments, but the code all checks out to me.
@@ -190,7 +190,10 @@ func TestStackCallCheckInstances(t *testing.T) { | |||
// when we know there are zero instances, which would be a non-nil | |||
// empty map. | |||
gotInsts, diags := call.CheckInstances(ctx, InspectPhase) | |||
assertNoDiags(t, diags) |
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.
Oh. 😆 yeah, makes sense for CheckInstances to check the for_each value. I'm surprised it didn't before.
internal/stacks/stackruntime/internal/stackeval/stack_call_test.go
Outdated
Show resolved
Hide resolved
Co-authored-by: Nick Fagerlund <nick.fagerlund@gmail.com>
Co-authored-by: Nick Fagerlund <nick.fagerlund@gmail.com>
Reminder for the merging maintainer: if this is a user-visible change, please update the changelog on the appropriate release branch. |
Fixes #
Target Release
1.8.x
Draft CHANGELOG entry
NEW FEATURES