-
Notifications
You must be signed in to change notification settings - Fork 1
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
Add GitHub actions to run unit tests and publish docker images #20 #23
Conversation
.github/workflows/ci.yml
Outdated
run: | | ||
if [ "${{ github.event.release.target_commitish }}" != "" ]; then | ||
BRANCH="${{ github.event.release.target_commitish }}" | ||
elif [[ $GITHUB_REF =~ pull ]]; then |
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 probably needs to be updated to check for "pull/" because it might hit this condition even if the branch name contains the work "pull". e.g. "pull-from-api"and I enable the trigger to run for other branches.
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 can probably use:
elif [[ $GITHUB_REF =~ pull ]]; then | |
elif [ "${{github.event.pull_request.number}}" != "" ]; then | |
BRANCH="${{github.event.pull_request.number}}" |
.github/workflows/ci.yml
Outdated
run: | | ||
if [ "${{ github.event.release.target_commitish }}" != "" ]; then | ||
BRANCH="${{ github.event.release.target_commitish }}" | ||
elif [[ $GITHUB_REF =~ pull ]]; then |
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 can probably use:
elif [[ $GITHUB_REF =~ pull ]]; then | |
elif [ "${{github.event.pull_request.number}}" != "" ]; then | |
BRANCH="${{github.event.pull_request.number}}" |
…20' into add-github-actions-to-services-#20
Closes #20
Please see the code comments for working.
Please looks at the condition I used to build/publish docker images, something could be wrong there. I'm still testing the PR and push(to develop) conditions - so far only tested on my branch