#pr-reviews - October 2024

May I get a review on this PR https://github.com/cloudposse/github-action-pre-commit/pull/26

If it's not too late to add this contribution to the AWS components, I can always hold of until after the migration (https://github.com/cloudposse/terraform-aws-components/pull/1143)

May I get a review on https://github.com/cloudposse/terraform-aws-components/pull/1149

Hey all, can I get a review of https://github.com/cloudposse/terraform-aws-dms/pull/36?

It's been two weeks and I'm still waiting for a review on https://github.com/cloudposse/build-harness/pull/388. How long do reviews typically take?

This is a two character change.

It's been a while since I last contributed, but I just created a small PR to allow specifying the environment for dns-delegated in the efs component. https://github.com/cloudposse/terraform-aws-components/pull/1164

Same for elasticache-redis: https://github.com/cloudposse/terraform-aws-components/pull/1165

Hey all, looks like https://github.com/cloudposse/terraform-aws-dms/pull/36 is ready to go. Can we merge it?

Small bugfix for the above PR to change an invalid parameter type: https://github.com/cloudposse/terraform-aws-dms/pull/37 Seems like my contributions are not needed anymore since another PR got merged for this.

Hey all, a bugfix for the PR I submitted last week: https://github.com/cloudposse/terraform-aws-dms/pull/38

Came across some interesting interplay between inspection levels and machine learning parameters in the WAF module today. Thought this code comment might be beneficial to the broader community utilizing these features: https://github.com/cloudposse/terraform-aws-waf/pull/104

And another to deprecate resource_collection_enabled in favor of extended_resource_collection_enabled in the Datadog Integration module: https://github.com/cloudposse/terraform-aws-datadog-integration/pull/66

Have been working on the Datadog Integration component recently and was wondering if I would be able to get a review on this PR to add additional IAM policies recommended by Datadog? https://github.com/cloudposse/terraform-aws-datadog-integration/pull/68

Please review valkey support for elasticache-redis terraform component

https://github.com/cloudposse/terraform-aws-components/pull/1170

Please review snapshot retention limit support for elasticache-redis terraform component

https://github.com/cloudposse/terraform-aws-components/pull/1171

https://github.com/cloudposse/terraform-aws-alb/pull/186 please 🙏