Skip to content

compliance: Reduce noisiness when checks were successful - #53515

Merged
MrSubidubi merged 3 commits into
mainfrom
better-messages
Apr 10, 2026
Merged

MrSubidubi merged 3 commits into
mainfrom
better-messages

Conversation

@MrSubidubi

Copy link
Copy Markdown
Member

This will ensure we do not post a second Slack message in case the first check was successful. We still run the second check, but do not notify

Release Notes:

  • N/A

@MrSubidubi
MrSubidubi requested a review from ConradIrwin April 9, 2026 12:19
@cla-bot cla-bot Bot added the cla-signed The user has signed the Contributor License Agreement label Apr 9, 2026
@zed-community-bot zed-community-bot Bot added the staff Pull requests authored by a current member of Zed staff label Apr 9, 2026

@ConradIrwin ConradIrwin 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.

nice

@MrSubidubi
MrSubidubi merged commit 4c63fb1 into main Apr 10, 2026
49 of 51 checks passed
@MrSubidubi
MrSubidubi deleted the better-messages branch April 10, 2026 16:39
@MrSubidubi

Copy link
Copy Markdown
Member Author

/cherry-pick preview

@zed-zippy

zed-zippy Bot commented Apr 10, 2026

Copy link
Copy Markdown
Contributor

piper-of-dawn pushed a commit to piper-of-dawn/zed that referenced this pull request Apr 25, 2026
…ies#53515)

This will ensure we do not post a second Slack message in case the first
check was successful. We still _run_ the second check, but do not notify

Release Notes:

- N/A
Zenor27 pushed a commit to Zenor27/zed that referenced this pull request Jul 4, 2026
…ies#53515)

This will ensure we do not post a second Slack message in case the first
check was successful. We still _run_ the second check, but do not notify

Release Notes:

- N/A
jonx pushed a commit to jonx/zed-aros that referenced this pull request Jul 17, 2026
…ies#53515)

This will ensure we do not post a second Slack message in case the first
check was successful. We still _run_ the second check, but do not notify

Release Notes:

- N/A
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla-signed The user has signed the Contributor License Agreement staff Pull requests authored by a current member of Zed staff

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants