RFC 0009: Adding looping functionality in CFN Template (Fn::For) #19
Conversation
| * The collection to iterate on. It should be a List. You can either in-place the collection itself or reference List type parameter values from Parameters section. | ||
| * `TemplateSnippet` | ||
| * The template snippet to replicate for each item in iteration. | ||
| * (Conditional) `LogicalId` |
There was a problem hiding this comment.
This should not be called LogicalId. Fn::For is not only allowed to be used in the context of resources. It's a generic function that could work in any array or JSON object (you even talk about it being used in the Output object in the description), so this parameter should be called something like ObjectKey and the description should be changed to not be specific to resources.
There was a problem hiding this comment.
I have a different opinion. Fn::For is supposed to work like a generic function, but LogicalId is required only when the function is used for replicating either Resource or Output, which is already described so in the section.
CloudFormation calls an identifier of Output also as "Logical Id". Therefore, I would prefer keeping the parameter name as LogicalId gives better impression of its characteristic being "Conditional": it has to be defined only for Resource or Output.
|
We had to recreate this PR because the fork it was based on was detached when we switched this repo to public. This RFC can now be tracked here: #75 |
Language Enhancement Request For Comment
This is a request for comments about a new Language Extensions intrinsic function
Fn::For. See TRACKING_ISSUE foradditional details.
Licensing
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.