Skip to content

feat: add slack notification for snyk failure - #411

Merged
jiahuii-amazon merged 2 commits into
awslabs:mainfrom
jiahuii-amazon:feat/add-snyk-failure-notification
Jan 30, 2026
Merged

feat: add slack notification for snyk failure#411
jiahuii-amazon merged 2 commits into
awslabs:mainfrom
jiahuii-amazon:feat/add-snyk-failure-notification

Conversation

@jiahuii-amazon

Copy link
Copy Markdown
Contributor

Describe your changes

Modify Snyk workflow to send a slack notification on failure.

Issue ticket number and link

Checklist before requesting a review

  • I updated CHANGELOG.MD with a description of my changes
  • If the change was to a module, I ran the code validation script (scripts/validate.sh)
  • If the change was to a module, I have added thorough tests
  • If the change was to a module, I have added/updated the module's README.md
  • If a module was added, I added a reference to the module to the repository's README.md
  • I verified that my code deploys successfully using seedfarmer apply

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@weklund

weklund commented Jan 29, 2026

Copy link
Copy Markdown
Contributor

Do you mind adding this change to the changelog?

Comment thread CHANGELOG.md

### **Added**

- added slack notification on failure in the snyk workflow

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

are github workflow changes needed in the changelog since they are not shipped to customer?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

think it's still useful for ppl who have access to this github repo

@jiahuii-amazon
jiahuii-amazon merged commit 770bd7d into awslabs:main Jan 30, 2026
@kwangaws kwangaws mentioned this pull request Feb 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants