#pr-reviews (2020-10)
Pull Request Reviews for Cloud Posse Projects
2020-10-01
Hi, I’ve just upgraded a module for TF 0.12 + 0.13 (mainly via the automated update commands) + updated dependency null-label https://github.com/cloudposse/terraform-aws-cloudwatch-flow-logs/pull/14
Hey Alex, thanks! This is a great start. We’ve been requiring that 0.11 to 0.12 upgrades include tests as it’s the only way to properly maintain this number of modules. So to get this merged, we’ll need to get an examples/complete example in place and our test harness / simple terratests in place.
As an example of this — here is a recent upgrade that I carried over the line yesterday: https://github.com/cloudposse/terraform-aws-iam-user/pull/9
Would you mind updating your PR to include similar updates?
2020-10-02
2020-10-04
2020-10-05
2020-10-06
2020-10-07
2020-10-13
https://github.com/cloudposse/terraform-aws-ecs-container-definition/pull/96/commits/b5b5d759cf7de6b67c534bb437d05b98f3773428 Small PR to remove not needed code