Skip to content
This repository has been archived by the owner on Apr 5, 2024. It is now read-only.

Create an issue on nightly CI run failure #301

Merged
merged 1 commit into from
May 29, 2022

Conversation

dario23
Copy link
Contributor

@dario23 dario23 commented May 22, 2022

Implements #299

fixes #299

@dario23 dario23 force-pushed the issue-on-ci-failure branch 3 times, most recently from 6958308 to 7311a9c Compare May 22, 2022 17:45
@dario23
Copy link
Contributor Author

dario23 commented May 22, 2022

create issue correctly skipped for the pull request build; i used the same github.event_name == 'schedule' as we do to skip setting up the latest nightly toolchain for non-nightly builds, so i hope that does its part, but not sure if we can explicitly test the other way before merging (and hopefully before the next nightly build failure).

.github/workflows/ci.yml Outdated Show resolved Hide resolved
.github/workflows/ci.yml Show resolved Hide resolved
@JohnTitor
Copy link
Member

Have you tested the script on your fork or somewhere else? Overall looks good but I'd like to make sure that variables in the script work fine.

@dario23 dario23 force-pushed the issue-on-ci-failure branch 4 times, most recently from 7628280 to 02cba1d Compare May 28, 2022 16:57
@dario23
Copy link
Contributor Author

dario23 commented May 28, 2022

alright, tested now, see also the latest force-push.

@JohnTitor
Copy link
Member

Awesome, thanks for testing!
@bors r+

@bors
Copy link
Contributor

bors commented May 29, 2022

📌 Commit 02cba1d has been approved by JohnTitor

@bors
Copy link
Contributor

bors commented May 29, 2022

⌛ Testing commit 02cba1d with merge 3884350...

@bors
Copy link
Contributor

bors commented May 29, 2022

☀️ Test successful - checks-actions
Approved by: JohnTitor
Pushing 3884350 to master...

@bors bors merged commit 3884350 into rust-lang:master May 29, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Open a new issue when cronjob fails
3 participants