Skip to content

[NDR-313] Add slack notification for workflow failures #880

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
megan-bower4 merged 16 commits into from
Nov 19, 2025
Merged

[NDR-313] Add slack notification for workflow failures #880

megan-bower4 merged 16 commits into from
Nov 19, 2025

Conversation

@megan-bower4
Copy link
Contributor

@megan-bower4 megan-bower4 commented Nov 11, 2025 •
edited
Loading

  • Temp check box

Description
Adding slack alerting on workflow failures for merge to main. In particular to automated testing and deployment.

robg-test
robg-test reviewed Nov 19, 2025
View reviewed changes
robg-test
robg-test previously approved these changes Nov 19, 2025
View reviewed changes
robg-test
robg-test reviewed Nov 19, 2025
View reviewed changes
@megan-bower4 megan-bower4 dismissed stale reviews from robg-test and jameslinnell via 58b89f5 November 19, 2025 13:10
@megan-bower4
[NDR-313] Testing slack notification
9d3c522
@megan-bower4
[NDR-313] Revert version
553a072
@megan-bower4
[NDR-313] Add workflow_call block to pass secret
ee73d15
@megan-bower4
[NDR-313] Add environment settings to workflow
b72741a
@megan-bower4
[NDR-313] Update env reference
06970f2
@megan-bower4
[NDR-313] Update slack message
cfd8de1
@megan-bower4
[NDR-313] Use json in slack message
6f26646
@megan-bower4
[NDR-313] Temporary test failure to test slack notification
fcfc477
@megan-bower4
[NDR-313] Move slack notifiacation to merge to main workflows only
cdb81fb
@megan-bower4
[NDR-313] Add slack notifiacation to other merge to main workflows
7672370
@megan-bower4
[NDR-313] Test branch name in message
751b663
@megan-bower4
[NDR-313] Remove False assertion
e90dffb
@megan-bower4
[NDR-313] Only alert on merge to main failures not pull requests
fa7116b
@megan-bower4
[NDR-313] Update link with actual workflow name
8990998
@megan-bower4
[NDR-313] Remove false assertion
52c685b
@megan-bower4
[NDR-313] Update step name
2eecc27
@github-actions
Copy link

github-actions bot commented Nov 19, 2025

Code security issues found

View full details here.

@sonarqubecloud
Copy link

sonarqubecloud bot commented Nov 19, 2025

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@megan-bower4 megan-bower4 merged commit b7ca0ab into main Nov 19, 2025
26 of 27 checks passed
@megan-bower4 megan-bower4 deleted the NDR-313 branch November 19, 2025 16:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jameslinnell jameslinnell jameslinnell approved these changes

@robg-test robg-test robg-test approved these changes

@SanjayChopraNHS SanjayChopraNHS Awaiting requested review from SanjayChopraNHS SanjayChopraNHS is a code owner automatically assigned from NHSDigital/access-store

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@megan-bower4 @jameslinnell @robg-test