This repository has been archived by the owner on Dec 7, 2023. It is now read-only.
[Fn::Sub
] Relax documented restrictions on String
parameter
#1306
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Issue #, if available:
aws-cloudformation/cfn-lint#2587
Description of changes:
Clarify the ability to use an
Fn::Transform
in theString
parameter of anFn::Sub
.Discussion
Looking for feedback on how to better integrate the use of
Fn::Transform
into the documentation as a whole. I think one way to do this is to relax some of the strict statements around the section I am editing here. If there is a better doc writer out there that wants to help me wordsmith the docs here, I am completely open to that.I feel that
Fn::Transform
is a bit more of an advanced CFN tool, and thus we may need to balance detail vs the need for simple documentation for more common features. However, I do strongly believe that the docs should not make strict claims about what is or is not possible in CFN. Meaning, the docs should maintain technical correctness.By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.