Skip to content

ci: Better Slack failures#9812

Merged
mergify[bot] merged 3 commits intomainfrom
thane/ci-better-slack-failures
Dec 1, 2022
Merged

ci: Better Slack failures#9812
mergify[bot] merged 3 commits intomainfrom
thane/ci-better-slack-failures

Conversation

@thanethomson
Copy link
Contributor

The primary aim of this PR is to improve the failure messages we get a little. Seeing the latest commit is often useless, and it's probably better to just get a direct link to a list of commits for that specific branch so we can see what's been committed since the previous nightly run.

This PR also makes one minor improvement in the use of the now-deprecated set-output syntax for workflows, upgrading it to use the new approach as per https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/


PR checklist

  • Tests written/updated, or no tests needed
  • CHANGELOG_PENDING.md updated, or no changelog entry needed
  • Updated relevant documentation (docs/) and code comments, or no
    documentation updates needed

Signed-off-by: Thane Thomson <connect@thanethomson.com>
Signed-off-by: Thane Thomson <connect@thanethomson.com>
@thanethomson thanethomson requested a review from a team December 1, 2022 12:44
@thanethomson thanethomson added the S:automerge Automatically merge PR when requirements pass label Dec 1, 2022
@mergify mergify bot merged commit 0952a99 into main Dec 1, 2022
@mergify mergify bot deleted the thane/ci-better-slack-failures branch December 1, 2022 17:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S:automerge Automatically merge PR when requirements pass

Projects

Status: Done/Merged

Development

Successfully merging this pull request may close these issues.

2 participants