Skip to content
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

HEAT-313 - implement Github workflows for test -> build -> deploy #245

Merged
merged 27 commits into from
Nov 12, 2024

Conversation

james-jdgtl
Copy link
Contributor

This is the initial release for deploying applications using Github Actions/ghcr.io rather than CircleCI/quay.io

The workflow has been validated on test projects - this is a protoype for Kotlin deployments.

# - '**'

workflow_dispatch:
inputs:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wonder if on a workflow dispatch we need all these options?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've reviewed the process for CircleCI and it carries out the workflow for all branches, but only builds and deploys on main. I've updated this PR to reflect that.

petergphillips
petergphillips previously approved these changes Oct 18, 2024
Copy link
Contributor

@petergphillips petergphillips left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Contributor

@Sandhya1874 Sandhya1874 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@mattops mattops merged commit f0e66c3 into main Nov 12, 2024
4 of 5 checks passed
@mattops mattops deleted the HEAT-313-kotlin-github-workflow branch November 12, 2024 14:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants