Skip to content
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.

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

Add Slack notification to Github workflows #329

Merged
merged 3 commits into from
Sep 10, 2024
Merged

Conversation

markgoddard
Copy link

Alert on failures for automatically triggered workflows. For manually
triggered workflows we rely on GitHub emails to the individual that
triggered the workflow.

See the "Workflow builder" approach in
https://github.com/slackapi/slack-github-action/.

Alert on failures for automatically triggered workflows. For manually
triggered workflows we rely on GitHub emails to the individual that
triggered the workflow.

See the "Workflow builder" approach in
https://github.com/slackapi/slack-github-action/.
@markgoddard markgoddard self-assigned this Sep 6, 2024
@markgoddard markgoddard requested a review from a team as a code owner September 6, 2024 14:20
@stackhpc-ci
Copy link
Contributor

stackhpc-ci commented Sep 9, 2024

Terraform Format and Style 🖌success

Terraform Initialization ⚙️success

Terraform Validation 🤖success

Validation Output

Success! The configuration is valid.


Terraform Plan 📖success

Show Plan

undefined

Pusher: @markgoddard, Action: pull_request, Working Directory: ``, Workflow: Terraform GitHub

@markgoddard
Copy link
Author

Added some docs, resolved merge conflicts, ready for review.

@markgoddard markgoddard merged commit f4114ae into main Sep 10, 2024
4 checks passed
@markgoddard markgoddard deleted the slack-notifications branch September 10, 2024 14:09
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