PePe Amengual
PePe Amengual
/test all
Please do the changes on the doc to the Readme.yaml file instead.
yes it does.
we are happy to receive PRs @njgerner
Out tests are passing so I do not think we are experiencing the same issue, have you looked at the new updates? @stormmore
This is a breaking change. You are modifying an existing variable and dissecting it into two which is not backward compatible. I can see what was done in the [terraform-aws-ecs-alb-service-task](https://github.com/cloudposse/terraform-aws-ecs-alb-service-task)...
that is needed by codepipeline and in our codepipeline readme : https://github.com/cloudposse/terraform-aws-ecs-codepipeline#example-buildspec it explains that is needed.
@razorsedge have you tried setting the module with `enabled = false` and then run terraform apply?