Skip to content

Setup slack notification for cron job#295

Merged
rahulporuri merged 1 commit into
masterfrom
dev/cron-job-slack-notification
Jul 14, 2021
Merged

Setup slack notification for cron job#295
rahulporuri merged 1 commit into
masterfrom
dev/cron-job-slack-notification

Conversation

@rahulporuri
Copy link
Copy Markdown
Contributor

@rahulporuri rahulporuri commented Jul 13, 2021

This PR sets up slack notification for the cron job. I manually triggered the cron job on this branch to ensure that the slack notification actually worked as expected. Ref https://github.com/enthought/apptools/actions/runs/1027301837

One more step towards enthought/ets#65.

Checklist

  • Add a news fragment if this PR is news-worthy for end users. (see docs/releases/README.rst)

	modified:   .github/workflows/ets-from-source.yml
@aaronayres35 aaronayres35 self-requested a review July 13, 2021 17:28
Copy link
Copy Markdown
Contributor

@aaronayres35 aaronayres35 left a comment

Choose a reason for hiding this comment

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

LGTM

@rahulporuri rahulporuri merged commit 7262188 into master Jul 14, 2021
@rahulporuri rahulporuri deleted the dev/cron-job-slack-notification branch July 14, 2021 05:48
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