-
Notifications
You must be signed in to change notification settings - Fork 724
Conversation
@danarbaugh would you be so kind to implement our suggestions? |
Co-authored-by: Philipp Trulson <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just one tiny change for the time property, rest looks good 👍
Co-authored-by: Philipp Trulson <[email protected]>
@der-eismann I believe this should be ready for a merge. |
This will be implemented via ekristen/aws-nuke#339 If you have a chance, please check it out and let us know if you run into an issues by opening an issue over on the fork. Please see the copy of the notice from the README about the deprecation of this project. Sven was kind enough to grant me access to help triage and close issues and pull requests that have already been addressed in the actively maintained fork. Some additional information is located in the welcome issue for more information. Caution This repository for aws-nuke is no longer being actively maintained. We recommend users to switch to the actively maintained fork of this project at ekristen/aws-nuke. |
This adds Polly support by removing the only resource the service manages outside of sound files output to local or S3; Lexicons.
Testing
Polly Lexicons were created using the Setup Bash Script included below. AWS Nuke then showed this successful removal within its logs with
PollyLexicon
included in the configuration.Setup Bash Script