Skip to content

Notify Slack when Craft Cloud reviews are requested - #197

Draft
timkelty wants to merge 6 commits into
3.xfrom
timkelty/notify-craft-cloud-reviews
Draft

Notify Slack when Craft Cloud reviews are requested#197
timkelty wants to merge 6 commits into
3.xfrom
timkelty/notify-craft-cloud-reviews

Conversation

@timkelty

@timkelty timkelty commented Aug 18, 2026

Copy link
Copy Markdown
Contributor

Calls the shared PR Slack notification workflow when the Craft Cloud team is requested for review, posting to #craft-cloud. The caller passes organization secrets with secrets: inherit.

Depends on craftcms/.github#106.

Copilot AI lite review requested due to automatic review settings August 18, 2026 13:49

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Adds a GitHub Actions workflow in craftcms/cloud that delegates “review requested” events to the shared craftcms/.github reusable workflow, so Craft Cloud team review requests can be forwarded to Slack.

Changes:

  • Introduces a new workflow that triggers on pull_request_targetreview_requested.
  • Calls the shared reusable workflow (craftcms/.github/.github/workflows/notify-craft-cloud-review.yml@v3) and forwards the Slack webhook secret.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread .github/workflows/notify-craft-cloud-review.yml Outdated
Copilot AI review requested due to automatic review settings August 18, 2026 13:54

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.

Comment thread .github/workflows/notify-craft-cloud-review.yml Outdated
Copilot AI review requested due to automatic review settings August 18, 2026 15:48

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

Suppressed comments (1)

.github/workflows/notify-craft-cloud-review.yml:13

  • The job currently runs for every review_requested event (including individual reviewer requests), which will create unnecessary workflow runs/noise. Add a job-level if: guard so the reusable workflow only runs when the craft-cloud team is the requested reviewer team.
  notify:
    uses: craftcms/.github/.github/workflows/notify-craft-cloud-review.yml@v3
    secrets:
      slack_webhook_url: ${{ secrets.CRAFT_CLOUD_SLACK_WEBHOOK_URL }}
Copilot AI review requested due to automatic review settings August 18, 2026 15:52

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

Copilot AI review requested due to automatic review settings August 18, 2026 16:14

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

Copilot AI review requested due to automatic review settings August 18, 2026 17:15
@timkelty
timkelty force-pushed the timkelty/notify-craft-cloud-reviews branch from 5086ee3 to c4a7020 Compare August 18, 2026 17:17

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

Copilot AI review requested due to automatic review settings August 18, 2026 17:18

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.

Comment thread .github/workflows/notify-craft-cloud-review.yml
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants