-
Notifications
You must be signed in to change notification settings - Fork 12
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
Explain what this does better than the native syntax? #5
Comments
Is this valid? I don't believe it's possible to specify "needs" on individual steps. I can find no documentation for it, and it doesn't work when I try it. Maybe I'm missing something. |
It doesn't work, You need to execute post-job code in a JavasScript action with |
@jorhett - name: Show VPN log
uses: webiny/[email protected]
if: always()
with:
run: sudo cat vpn.log Thanks to @adrians5j for publishing this action! |
I think your solution is working if you create a plain simple github action workflow. Check out my repo where I successfully use this action: https://github.com/swissbuechi/action-openvpn-client |
It's hard to grasp why this action exists, as this feature set is built into Actions today. If this provides a value not provided by GitHub Actions syntax you should add it to the documentation.
The documentation's example runs exactly the same, and much faster, written as the following:
The text was updated successfully, but these errors were encountered: