Skip to content

ci: post merges and releases to Slack - #48

Merged
gorandodig merged 1 commit into
devfrom
ci/slack-notifications
Nov 4, 2026
Merged

gorandodig merged 1 commit into
devfrom
ci/slack-notifications

Conversation

@gorandodig

Copy link
Copy Markdown
Member

Wires pyro into Slack the same way as the other plainlang repos. It uses the org SLACK_BOT_TOKEN secret and SLACK_CHANNEL_ID variable, and skips Slack when the variable is not shared with this repo.

  • Merges into dev post one message. Release PRs are skipped because release.yml posts those. The trigger is pull_request, not pull_request_target, which runs from main and so lags dev until a stable release. Fork PRs get no secrets, so their merges post nothing; no fork PR has been merged here so far.
  • Releases post a start notice from the release job. A separate notify job replies in its thread with the result. Failures are also broadcast to the channel, and success adds a ✅ reaction to the start notice. If there is no start notice, results post to the channel.

Same setup as the other plainlang repos: org bot token and channel,
release results threaded under a start notice, failures broadcast,
and a check-mark reaction on success.

Merges use pull_request, not pull_request_target. The latter runs
from main, which lags dev until a stable release.
@coderabbitai

coderabbitai Bot commented Nov 3, 2026 •

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 80240a4b-74d7-4965-93f5-bca5c2449f9c

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • Trigger review
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Tip

🎯 Customize your risk assessments

Guide how CodeRabbit assesses merge risk and security architecture risk. Highlight sensitive changes and explain your security boundaries using Merge Risk Instructions and Security Risk Instructions in your review settings. Security architecture risk assessments require the Advanced plan or above.

✅ Automatic approvals, with your rules

Let CodeRabbit approve pull requests once required threads are resolved, pre-merge checks pass, and your risk limits are met. Enable Automatic approvals in review settings, with overrides for teams or file paths.

@gorandodig
gorandodig marked this pull request as ready for review November 4, 2026 14:57
@gorandodig
gorandodig merged commit 4ccfe44 into dev Nov 4, 2026
7 checks passed
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