-
Notifications
You must be signed in to change notification settings - Fork 63.1k
Add instructions how to use status check outside if conditionals #39464
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
Conversation
Thanks for opening this pull request! A GitHub docs team member should be by to give feedback soon. In the meantime, please check out the contributing guidelines.
How to review these changes 👓Thank you for your contribution. To review these changes, choose one of the following options: A Hubber will need to deploy your changes internally to review. Table of review linksNote: Please update the URL for your staging server or codespace. The table shows the files in the
Key: fpt: Free, Pro, Team; ghec: GitHub Enterprise Cloud; ghes: GitHub Enterprise Server 🤖 This comment is automatically generated. |
@arxeiss Thanks for opening an issue and PR about this! I'm glad you found a function that made your life easier. I'll definitely look into why it's not documented and whether it should be!
7760b72
to
6926674
Compare
@Sharra-writes any updates on this? In my opinion this is still valid update.
If my PR is missing anything, let me know
@arxeiss The only update is that I have a really long queue of questions about Actions, and the team is responding slowly in part because people have been/are on vacation, and in part because Actions is complicated with a lot of specialization. This hasn't fallen off my list.
Understand. Thank you for update
This comment was marked as spam.
This comment was marked as spam.
Co-authored-by: Jacob Wallraff <thyeggman@github.com>
Thanks very much for contributing! Your pull request has been merged 🎉 You should see your changes appear on the site in approximately 24 hours. If you're looking for your next contribution, check out our help wanted issues ⚡
Why:
Closes: #39463
We want to send Slack message when job succeed, but also fails. And we had 2 action steps with same setup, except the message. By coincidence I have found that you can use
job.status
too. But it is not mentioned in the Status check functions. So I simply thought it is not possible at all.After my finding, I was able to convert this
into
What's being changed (if available, include any code snippets, screenshots, or gifs):
Adding 1 more paragraph to Status check functions with link to https://docs.github.com/en/actions/reference/contexts-reference#job-context describing how to access same info like
success()
/failure()
outside if conditionalsCheck off the following: