Skip to content

ACP/nominated Zulip notifications - #901

Merged
nia-e merged 1 commit into
rust-lang:mainfrom
clarfonthey:acp-channel
Sep 30, 2026
Merged

nia-e merged 1 commit into
rust-lang:mainfrom
clarfonthey:acp-channel

Conversation

@clarfonthey

Copy link
Copy Markdown
Contributor

Implements #893.

@Urgau Urgau left a comment •

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This setup really seems like a poor man version of T-compiler major change system. Wouldn't it be better to use it instead?

It would the avantage of supporting auto-closing and concerns out of the box.

View changes since this review

Comment thread triagebot.toml Outdated
Comment thread triagebot.toml Outdated
@clarfonthey
clarfonthey force-pushed the acp-channel branch 2 times, most recently from 34d46ef to 5749995 Compare September 30, 2026 13:40
@clarfonthey clarfonthey changed the title ACP Zulip notifications ACP/nominated Zulip notifications Sep 30, 2026
@clarfonthey
clarfonthey force-pushed the acp-channel branch 2 times, most recently from 2ab84de to eb8a2fb Compare September 30, 2026 16:29
@nia-e

nia-e commented Sep 30, 2026

Copy link
Copy Markdown
Member

we did consider it (discussed last meeting) but there's a fair few differences in protocol and the mcp system is a bit too heavyweight for us, when acps instead should be very light ^^

@nia-e
nia-e merged commit 95696b1 into rust-lang:main Sep 30, 2026
1 check passed
@clarfonthey
clarfonthey deleted the acp-channel branch September 30, 2026 22:06
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.

3 participants