FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Remove comment-on-release workflow by Kludex · Pull Request #2985 · modelcontextprotocol/python-sdk · GitHub

Remove comment-on-release workflow - #2985

Merged
Kludex merged 1 commit into
mainfrom
drop-comment-on-release-workflow
Jun 26, 2026
Merged

Remove comment-on-release workflow#2985
Kludex merged 1 commit into
mainfrom
drop-comment-on-release-workflow

Conversation

Kludex commented Jun 26, 2026

Copy link
Copy Markdown
Member

Drops .github/workflows/comment-on-release.yml, which commented on every PR included in a release whenever a release was published.

AI Disclaimer

This PR was developed with the assistance of either Claude or Codex. I've reviewed and verified the changes.

cubic-dev-ai 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.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

No issues found across 1 file

Re-trigger cubic

Kludex enabled auto-merge (squash) June 26, 2026 08:59

claude Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

LGTM — straightforward removal of the release-comment workflow with no remaining references.

Extended reasoning...

Overview

This PR deletes a single file, .github/workflows/comment-on-release.yml, a GitHub Actions workflow that posted a comment on every PR included in a release whenever a release was published. No other files are touched, and a repo-wide search confirms nothing else references this workflow (it is not invoked via workflow_call or mentioned in docs/scripts).

Security risks

None. Removing a workflow only reduces the repository's automation surface; the deleted workflow held pull-requests: write permission, so its removal slightly shrinks the set of jobs running with write tokens. There is no application code, dependency, or release/publish pipeline change.

Level of scrutiny

Low. This is a self-contained deletion of a developer-convenience CI workflow — it does not affect the SDK's runtime behavior, test suite, packaging, or release publishing. The intent (stop auto-commenting on PRs at release time) is clear from the title and description, and a deletion cannot introduce logic bugs.

Other factors

The bug-hunting system found no issues, and there are no outstanding reviewer comments on the PR. The change is trivially reversible if the team later wants the release comments back.

Kludex merged commit e9cd169 into main Jun 26, 2026
35 checks passed
Kludex deleted the drop-comment-on-release-workflow branch June 26, 2026 09:00
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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.

2 participants


Back | FazBrowse Home | New Git URL