Marking as feature request. @sanathkr what's your thoughts on this?
Great idea. Let's do it. PRs please :-)
@kyleknap @sanathkr It looks like this won't be so easy to implement as you guys have made assumptions in the current code that won't work with AWS::Include
, which can literally be placed anywhere.
Any luck? This would be a huge improvement of the CloudFormation workflow, which is the main obstacle of people adopting CloudFormation widely!
Any update on this issue? It would be a fantastic feature to have.
This would be a huge help for the use case from this example: https://github.com/awslabs/serverless-application-model/blob/master/examples/2016-10-31/api_swagger_cors/template.yaml
Good Morning!
We're closing this issue here on GitHub, as part of our migration to UserVoice for feature requests involving the AWS CLI.
This will let us get the most important features to you, by making it easier to search for and show support for the features you care the most about, without diluting the conversation with bug reports.
As a quick UserVoice primer (if not already familiar): after an idea is posted, people can vote on the ideas, and the product team will be responding directly to the most popular suggestions.
We鈥檝e imported existing feature requests from GitHub - Search for this issue there!
And don't worry, this issue will still exist on GitHub for posterity's sake. As it鈥檚 a text-only import of the original post into UserVoice, we鈥檒l still be keeping in mind the comments and discussion that already exist here on the GitHub issue.
GitHub will remain the channel for reporting bugs.
Once again, this issue can now be found by searching for the title on: https://aws.uservoice.com/forums/598381-aws-command-line-interface
-The AWS SDKs & Tools Team
This entry can specifically be found on UserVoice at: https://aws.uservoice.com/forums/598381-aws-command-line-interface/suggestions/33168370-allow-aws-cloudformation-package-to-work-with-lo
Based on community feedback, we have decided to return feature requests to GitHub issues.
this would be a major benefit as @lafiosca mentions the exact same problem i'm trying to solve :)
@sanathkr @lafiosca @georgealton @nikolay
I just submit a PR that would add this for inline swagger definitions on AWS::Serverless::Api resources. Adding it for arbitrary resources would be much more difficult, it seems this is the main use case though
Update: I have implemented this for arbitrary include transforms, not sure about my implementation approach, here is the PR, I will close the previous one if we decide to go ahead with this https://github.com/aws/aws-cli/pull/3454
@kyleknap @sanathkr @joguSD @stealthycoin how do i go about getting my PR for this reviewed?
https://github.com/aws/aws-cli/pull/3454
I also want this functionality. Let's get this merged!
Looks like the PRis merged but the Issue is still open. Need to close it.
Closing this as #3454 has been merged.
Most helpful comment
Great idea. Let's do it. PRs please :-)