Skip to content

[CI] Skip Discord alerts for draft PRs#498

Merged
CaseyLabs merged 4 commits intomasterfrom
ci/update-discord-alert
Apr 20, 2026
Merged

[CI] Skip Discord alerts for draft PRs#498
CaseyLabs merged 4 commits intomasterfrom
ci/update-discord-alert

Conversation

@CaseyLabs
Copy link
Copy Markdown
Collaborator

@CaseyLabs CaseyLabs commented Apr 18, 2026

Summary

  • skip discord-notify when the pull request is still a draft
  • listen for ready_for_review so the notification runs when a PR leaves draft state

Why? Because draft PRs are usually to run CI tests for validation, and we don't want to annoy reviewers with an incomplete PR until it's ready,

Validation

  • actionlint .github/workflows/discord-notify.yml
  • yamllint .github/workflows/discord-notify.yml

@CaseyLabs CaseyLabs marked this pull request as ready for review April 20, 2026 16:24
@CaseyLabs CaseyLabs requested a review from Volte6 as a code owner April 20, 2026 16:24
@CaseyLabs CaseyLabs merged commit c44088b into master Apr 20, 2026
7 checks passed
@CaseyLabs CaseyLabs deleted the ci/update-discord-alert branch April 20, 2026 16:47
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