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 issue alert on failed scheduled GHA #1781

Merged
merged 7 commits into from
Aug 15, 2022
Merged

Add issue alert on failed scheduled GHA #1781

merged 7 commits into from
Aug 15, 2022

Conversation

msaroufim
Copy link
Member

@msaroufim msaroufim commented Aug 5, 2022

Description

Adds an alert on failed Github scheduled Github Action. This is the exact same trick used in pytorch/examples

Code borrowed from: https://github.com/pytorch/examples/blob/main/.github/workflows/main.yml#L34

And a test showing it works pytorch/examples#1020

@msaroufim msaroufim requested a review from lxning August 5, 2022 17:58
@codecov
Copy link

codecov bot commented Aug 5, 2022

Codecov Report

❗ No coverage uploaded for pull request base (master@7fb5c92). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #1781   +/-   ##
=========================================
  Coverage          ?   45.38%           
=========================================
  Files             ?       64           
  Lines             ?     2591           
  Branches          ?       60           
=========================================
  Hits              ?     1176           
  Misses            ?     1415           
  Partials          ?        0           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@msaroufim msaroufim merged commit e327e6e into master Aug 15, 2022
@msaroufim msaroufim deleted the actionalert branch August 15, 2022 21:00
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.

None yet

3 participants