Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

allow empty dynamic blocks with ConfigModeAttr #21549

Merged
merged 1 commit into from
Jun 1, 2019

Conversation

jbardin
Copy link
Member

@jbardin jbardin commented May 31, 2019

If a dynamic block is evaluated zero times, the body content will
contain 0 blocks. Allow the probe for ConfigModeAttr to accept that no
blocks with a matching attribute should still be converted to a block if
they are called with dynamicExpand.

Fixes #21373

If a dynamic block is evaluated zero times, the body content will
contain 0 blocks. Allow the probe for ConfigModeAttr to accept that no
blocks with a matching attribute should still be converted to a block if
they are called with dynamicExpand.
@jbardin jbardin requested a review from a team May 31, 2019 21:31
@jbardin jbardin merged commit ebc2e5f into master Jun 1, 2019
@jbardin jbardin deleted the jbardin/empty-dynamic-attr branch June 1, 2019 17:01
@ghost
Copy link

ghost commented Jul 25, 2019

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.

@ghost ghost locked and limited conversation to collaborators Jul 25, 2019
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

0.12 "dynamic" blocs fail with empty iterator
2 participants