New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

feat(module): bump notification module and replace accounts id var #62

Open

shanilhirani wants to merge 1 commit into main from feat/sa-519

shanilhirani commented Jan 14, 2025 •

edited

Loading

As per title,

this updates the notification module to 2.0 and replaces the variable


          feat(module): bump notification module and replace accounts id var

20e1354

github-actions bot commented Jan 14, 2025

Pull Request Review Status

🖌 Terraform Format and Style: success
🔍 Terraform Linting: success
👮 Terraform Security Check: success
👮 Terraform Security Checkov: success
🔘 Terraform Tests: success
🔧 Terraform Initialisation: success
🤖 Terraform Validation: success
🤖 Terraform Example Validation: success
📖 Terraform Documentation: success
🔖 Commitlint: success

Working Directory: .
Pusher: @shanilhirani, Action: pull_request
Workflow Run Link: https://github.com/appvia/terraform-aws-alarm-baseline/actions/runs/12768533947

shanilhirani requested review from m13t, gambol99, KashifSaadat, danny-molnar, richcto and PriyeshM1

January 14, 2025 14:04

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

m13t Awaiting requested review from m13t

gambol99 Awaiting requested review from gambol99

KashifSaadat Awaiting requested review from KashifSaadat

danny-molnar Awaiting requested review from danny-molnar

richcto Awaiting requested review from richcto

PriyeshM1 Awaiting requested review from PriyeshM1

At least 1 approving review is required to merge this pull request.

Labels

None yet