Skip to content

Send error notification if issue occurs during publish - #156

Merged
BrandonChanSkybox merged 42 commits into
mainfrom
notifyOnFail
Aug 18, 2026
Merged

BrandonChanSkybox merged 42 commits into
mainfrom
notifyOnFail

Conversation

@BrandonChanSkybox

@BrandonChanSkybox BrandonChanSkybox commented Aug 5, 2026 •

Copy link
Copy Markdown
Contributor

Sends an error notification if issue occurs during publish on main

@BrandonChanSkybox
BrandonChanSkybox marked this pull request as ready for review August 10, 2026 22:40

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

Barebones and simple! Hopefully we can set up the main-only secret easily

Comment thread .github/workflows/release-changes.yml Outdated
echo "Could not find secret"
else
short_sha="${GITHUB_SHA::6}"
curl -X POST -H 'Content-type: application/json' --data "{\"text\":\"*Open Source Pipeline Failure!*\n>An issue occurred in commit: <${{github.server_url}}/${{github.repository}}/commit/${{github.sha}}|${short_sha}...>\"}" $test_secret

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.

Please update this to be a shared utility script for pipelines instead of a one-off bash script.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Updated and added a new script, notify-on-error.sh that is called in the yml file

@BrandonChanSkybox
BrandonChanSkybox merged commit c723e68 into main Aug 18, 2026
6 checks passed
@BrandonChanSkybox
BrandonChanSkybox deleted the notifyOnFail branch August 18, 2026 22:42
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