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

ci: reuse shared AI Slack notification summaries#638
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
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