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?