Skip to content

ci: reuse shared AI Slack notification summaries - #638

Closed
zhongkechen wants to merge 2 commits into
mainfrom
ci/use-shared-ai-slack-summary
Closed

zhongkechen wants to merge 2 commits into
mainfrom
ci/use-shared-ai-slack-summary

Conversation

@zhongkechen

Copy link
Copy Markdown
Contributor

Summary

  • grant the reusable Slack workflow contents: read and models: read
  • pin the AI summary implementation from aws/aws-durable-execution-ci#14
  • add a regression test that keeps this repository as a thin caller

Depends on aws/aws-durable-execution-ci#14. The workflow pin currently references that PR's head commit and must be updated to the merged commit before this PR is marked ready.

Testing

  • hatch run test:all .github/scripts/tests/test_build_lambda_layer.py .github/scripts/tests/test_notify_workflow.py .github/scripts/tests/test_parse_sdk_branch.py (7 tests)
  • hatch fmt --check .github/scripts/tests/test_notify_workflow.py
  • actionlint on .github/workflows/notify.yml and .github/workflows/test-parser.yml
  • git diff --check

@zhongkechen
zhongkechen deleted the ci/use-shared-ai-slack-summary branch August 13, 2026 17:55

This branch had an error being deployed

1 failed deployment
ai-pr-review — 887feaa5 Deployed Aug 13, 2026 by zhongkechen via ai-pr-review / Approve AI PR review #189
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.

1 participant