-
Notifications
You must be signed in to change notification settings - Fork 1
Github deploy action #151
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
Github deploy action #151
Conversation
| default: 'staging' | ||
| type: choice | ||
| options: | ||
| - production |
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 is pretty much redundant and not used, but I have kept it anyway
| uses: shimataro/[email protected] | ||
| with: | ||
| key: ${{ secrets.SSH_KEY }} | ||
| known_hosts: ${{ secrets.KNOWN_HOSTS }} |
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.
I have added these to the github secrets
.github/workflows/deploy.yml
Outdated
| inputs: | ||
| environment: | ||
| required: true | ||
| default: 'staging' |
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.
Is it intentional to set the default to staging even though staging is not one of the available choices?
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.
Updated
hemant10yadav
left a comment
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.
Looks good. Thanks!
It's been hard to deploy from Mac (with Apple Silicon), this should make it easy to deploy. I have adapted this from commcare-connect github deploy action. Currently, it doesn't have the slackbot integration. PR for that is seperate #152