-
Notifications
You must be signed in to change notification settings - Fork 64
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
gh-actions: fix release workflow #1332
gh-actions: fix release workflow #1332
Conversation
.github/workflows/deploy.yml
Outdated
- name: Validate TARGET_DIR | ||
run: echo "TARGET_DIR=$TARGET_DIR" | ||
|
||
### Exit if ref is not develop branch or release tag |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This could probably be moved up the chain, since it doesn't require anything else to run (I don't think?). There's no point getting this far if we're just going to exit...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We wanted to run the build and artifact upload before exiting (on each branch that triggers the workflow). But you are right, we could probably exit before setting the target_dir variable.
You're a star, @musicEnfanthen ! GH Actions are not fun to write. |
@ahankinson If you are OK with the latest changes, you can merge this PR. It goes to the release branch. |
This PR tries to fix the failing release workflow. It imrpoves and simplifies the exit and if conditions, and adds some validation steps that will make it easier to debug failing workflows in the future.