Closed Kostavro closed 2 years ago
Thank you for the feedback @antonbabenko . Obviously I also missed running pre-commits properly before. Pushed new commits to implement your suggestions.
This PR is included in version 3.5.0 :tada:
I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues. If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.
Description
Adding new output for certificate status. Supported attribute as mentioned in https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/acm_certificate#status .
Motivation and Context
Fixes https://github.com/terraform-aws-modules/terraform-aws-acm/issues/109
Breaking Changes
No breaking changes, just adding a new output.
How Has This Been Tested?
examples/*
to demonstrate and validate my change(s)examples/*
projectspre-commit run -a
on my pull request