Nuru

Results 226 comments of Nuru

> > > @aknysh What work and testing do you need to complete the PR? > > > The links you've sent refer to the master status of the two...

@jchanam I am hesitant to approve this PR because, while it resolved the confusion you experienced, I worry that it will add confusion to someone else. Please see my comment...

@jchanam Sorry if this is confusing, but it follows [the pattern that optional data is provided in a list of zero or one items](https://github.com/cloudposse/terraform-aws-security-group/wiki/Naming-Conventions,-Deprecating-Inputs,-Optional-Inputs#optional-inputs). The userdata can be a multi-line...

Is this still a problem with the current module? Is #31 still needed?

Not actually fixed by #31 or #44, plus those changes introduced a new bug: #45 I propose the easy fix for this which is to remove the option to filter...

This has been implemented in [terraform-aws-dynamic-subnets](https://github.com/cloudposse/terraform-aws-dynamic-subnets)

I believe this is a breaking change in the sense that it would require Terraform version >= 1.1, right? We are still, at the moment, allowing Terraform 0.14 for most...