Skip to content

feat(notify-pipeline-complete): support custom state and target sha - #817

Merged
danadajian merged 2 commits into
mainfrom
feat/notify-pipeline-complete-state
Oct 29, 2026
Merged

danadajian merged 2 commits into
mainfrom
feat/notify-pipeline-complete-state

Conversation

@danadajian

@danadajian danadajian commented Oct 29, 2026 •

Copy link
Copy Markdown
Contributor

Adds two optional inputs to notify-pipeline-complete so a release-train consumer can report a crashed train accurately: it marks that train's own deployment as failed, even when a newer deployment exists in the environment, and sets a failing Pipeline Status on open and queued PRs.

  • state: the state applied to the deployment status and to the Pipeline Status commit statuses on merge-queue SHAs and open PR head SHAs. Defaults to success, so existing callers are unaffected.
  • sha: when provided, listDeployments is filtered by that SHA so the deployment for that commit is marked, matching set-deployment-status. When omitted, the newest deployment in the environment is marked as before. Commit-status targeting is unchanged.
  • Tests cover a failure state on both the merge-queue and open-PR paths, and confirm listDeployments receives sha only when it is provided.
  • Both inputs are documented in the README.

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@danadajian
danadajian requested a review from a team as a code owner October 29, 2026 14:23
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@danadajian danadajian changed the title feat(notify-pipeline-complete): support custom state input feat(notify-pipeline-complete): support custom state and target sha Oct 29, 2026
@danadajian
danadajian merged commit 7635899 into main Oct 29, 2026
5 checks passed
@danadajian
danadajian deleted the feat/notify-pipeline-complete-state branch October 29, 2026 14:38
@eg-oss-ci

Copy link
Copy Markdown

🎉 This PR is included in version 1.91.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants