Skip to content

Julenmendieta/removeTelegram#30

Merged
julenmendieta merged 2 commits into
mainfrom
julenmendieta/removeTelegram
May 18, 2026
Merged

Julenmendieta/removeTelegram#30
julenmendieta merged 2 commits into
mainfrom
julenmendieta/removeTelegram

Conversation

@julenmendieta

@julenmendieta julenmendieta commented May 18, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This PR removes Telegram CI notifications from the mark-stable workflow and replaces them with Slack notifications, matching the pattern already used in build.yaml. A changeset entry is also added to record a patch bump across all rarefaction packages.

  • Removes notify-telegram: true and the two Telegram secrets (TELEGRAM_NOTIFICATION_TARGET, TELEGRAM_API_TOKEN) from .github/workflows/mark-stable.yaml, replacing them with SLACK_BOT_TOKEN and SLACK_CHANNEL — bringing this workflow in line with build.yaml.
  • Adds a changeset file marking a patch release for all six rarefaction packages with the message "Remove telegram".

Confidence Score: 5/5

Safe to merge — the change swaps two Telegram secrets for two Slack secrets in a manually-triggered workflow, mirroring the notification setup already live in build.yaml.

Both changed files are straightforward: the workflow change is a direct like-for-like substitution of notification secrets, and the changeset file correctly documents all affected packages. No logic, build steps, or publishing behavior is altered.

No files require special attention.

Important Files Changed

Filename Overview
.github/workflows/mark-stable.yaml Removes Telegram notification config and replaces with Slack secrets, consistent with the existing build.yaml pattern.
.changeset/cold-regions-beg.md New changeset file registering a patch bump for all six rarefaction packages with the "Remove telegram" description.

Sequence Diagram

sequenceDiagram
    participant GHA as GitHub Actions
    participant WF as block-mark-stable.yaml@v4
    participant Slack as Slack (SLACK_BOT_TOKEN)

    GHA->>WF: workflow_dispatch trigger
    WF->>WF: build / mark stable
    WF->>Slack: post notification to SLACK_BLOCKS_CI_CHANNEL
    Note over WF,Slack: Previously sent to Telegram (TG_CHANNEL_MIBUILDS)<br/>Now uses Slack — consistent with build.yaml
Loading

Reviews (1): Last reviewed commit: "Changeset" | Re-trigger Greptile

@gemini-code-assist gemini-code-assist Bot 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.

Code Review

This pull request introduces a changeset file to track the removal of 'telegram' functionality. The reviewer noted that the PR lacks the corresponding code changes and suggested updating the versioning from 'patch' to 'minor' to better reflect the nature of the change, providing a code suggestion for the version bump.

Comment thread .changeset/cold-regions-beg.md
@julenmendieta julenmendieta merged commit 5244cbb into main May 18, 2026
9 checks passed
@julenmendieta julenmendieta deleted the julenmendieta/removeTelegram branch May 18, 2026 14:39
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